nx icon indicating copy to clipboard operation
nx copied to clipboard

Add throw of error if a dependency is not in the graph.

Open richtera opened this issue 2 years ago • 5 comments

Ref https://github.com/nrwl/nx/issues/11077

Add error throw in the case a dependency is not found in the project graph.

NOTES: For some reason the azure CI config scripts fail because of master vs main base branch; I didn't change any of that. I executed jest with -u to update some of the snapshots, but I wasn't sure if that's something to do as part of a PR or not. It seems someone was running -u with the main branch being main and not master like on github.

image

Various other tests also fail image There is no /root/dist/apps folder under normal circumstances and definitely not user read/writable.

Creating a folder at /root/dist/apps and chmod each segment to 0777 does work.

richtera avatar Jul 08 '22 16:07 richtera

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 964de53faeebf30209e9f4115b7c6455d22f2ada. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch


🟥 Failed Commands
nx-cloud record -- yarn nx format:check --base=a34402fd9e99991bff8c24a150fb83916702ecbb --head=964de53faeebf30209e9f4115b7c6455d22f2ada
✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

nx-cloud[bot] avatar Jul 08 '22 16:07 nx-cloud[bot]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
nx-dev ✅ Ready (Inspect) Visit Preview Jul 8, 2022 at 4:23PM (UTC)

vercel[bot] avatar Jul 08 '22 16:07 vercel[bot]

For some reasons there are access denied errors on the agent builds other than macOS.

richtera avatar Jul 08 '22 22:07 richtera

What happened to this pr? These builds are failing because of some permission issue it seems.

richtera avatar Jul 17 '22 08:07 richtera

Is there something I can do to have this go anywhere?

richtera avatar Jul 26 '22 22:07 richtera

Applying in #11652, since I don't have permissions to push changes to fork and some were required for CI to pass.

AgentEnder avatar Aug 19 '22 21:08 AgentEnder

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

github-actions[bot] avatar Mar 18 '23 01:03 github-actions[bot]