fix(core): ignore angular.json projects if @nrwl/angular is not insta…

[FrozenPandaz]

…lled

Current Behavior

Most workspaces use Nx and Angular together. However, workspaces which use Nx and Angular CLI entirely independently have a conflict in some package names.

Expected Behavior

Projects that use Nx and Angular entirely independently do not have @nrwl/angular installed and should not read projects from angular.json

Related Issue(s)

Fixes #

[vercel]

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

Name	Status	Preview	Updated
nx-dev	✅ Ready (Inspect)	Visit Preview	Dec 14, 2022 at 7:25PM (UTC)

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit bda2a75. 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 depcheck --no-dte

✅ Successfully ran 18 targets

• nx affected --target=e2e --base=486516ea1384d15ea74fe8334687355d1aa98f51 --head=bda2a75b0f46337c1c6ee0c858115662bcf64fb8 --exclude=e2e-storybook,e2e-storybook-angular,e2e-react-native,e2e-detox --parallel=1
• nx depcheck --no-dte
• nx affected --target=lint --base=486516ea1384d15ea74fe8334687355d1aa98f51 --head=bda2a75b0f46337c1c6ee0c858115662bcf64fb8 --parallel=3
• nx lint --no-dte
• nx check-lock-files --no-dte
• nx check-commit --no-dte
• nx affected --target=build --base=486516ea1384d15ea74fe8334687355d1aa98f51 --head=bda2a75b0f46337c1c6ee0c858115662bcf64fb8 --parallel=3
• nx affected --target=test --base=486516ea1384d15ea74fe8334687355d1aa98f51 --head=bda2a75b0f46337c1c6ee0c858115662bcf64fb8 --parallel=1
• nx-cloud record -- nx format:check --base=486516ea1384d15ea74fe8334687355d1aa98f51 --head=bda2a75b0f46337c1c6ee0c858115662bcf64fb8
• nx check-imports --no-dte
• nx documentation --no-dte
• nx affected --target=lint --base=486516ea1384d15ea74fe8334687355d1aa98f51 --head=97b622cb9fe4f8f6c295800a124bad2cf1e896af --parallel=3
• nx lint --no-dte
• nx check-lock-files --no-dte
• nx check-commit --no-dte
• nx-cloud record -- nx format:check --base=486516ea1384d15ea74fe8334687355d1aa98f51 --head=97b622cb9fe4f8f6c295800a124bad2cf1e896af
• nx check-imports --no-dte
• nx documentation --no-dte

---

Sent with 💌 from NxCloud.

[AgentEnder]

Some debate on whether require.resolve vs reading package.json and checking deps is better for this check.

This could potentially have a false positive if there was @nrwl/angular found in one of the dependency chains of another installed library, but that should be relatively rare. In any case, the behavior exhibited would be the currently existing behavior, so its a low impact.

[github-actions]

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.