docs(angular): document PNPM workaround for ngcc

[isaacplmann]

Fix from #18100

Current Behavior

Currently, https://nx.dev/recipes/other/setup-incremental-builds-angular#requirements still warns that ngcc doesn't support PNPM.

Expected Behavior

While ngcc doesn't support PNPM out-of-the-box, PNPM has since added a node-linker=hoisted mode that creates flat node_modules without symlinks, similar to the node_modules created by NPM, allowing ngcc to work with PNPM (angular/angular#50735).

Unfortunately, using node-link=hoisted breaks @parcel/watcher that is used by NX (#16319 and parcel-bundler/watcher#142). A workaround is to explicitly declare node-gyp-build in package.json, e.g.

{
  ...
  "devDependencies": {
    ...
    "node-gyp-build": "4.6.0",
    ...
  }
  ...
}

This PR documents the workarounds to use ngcc with PNPM.

p.s. Huge thanks to @DzmVasileusky for discovering the workarounds.

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
nx-dev	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 7, 2023 1:39am

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 0266dae. 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

---

⌛ The following target is in progress

• nx affected --target=lint --base=1572749f4a676d731ab761a045ff35426ab62a79 --head=0266dae16893c0975870b6b9e85149cbfce73770 --parallel=3

✅ Successfully ran 7 targets

• nx affected --target=e2e --base=1572749f4a676d731ab761a045ff35426ab62a79 --head=0266dae16893c0975870b6b9e85149cbfce73770 --parallel=1
• nx affected --target=test --base=1572749f4a676d731ab761a045ff35426ab62a79 --head=0266dae16893c0975870b6b9e85149cbfce73770 --parallel=1
• nx affected --target=build --base=1572749f4a676d731ab761a045ff35426ab62a79 --head=0266dae16893c0975870b6b9e85149cbfce73770 --parallel=3
• nx run-many -t check-imports check-commit check-lock-files check-codeowners documentation --parallel=1 --no-dte
• nx-cloud record -- nx format:check --base=1572749f4a676d731ab761a045ff35426ab62a79 --head=0266dae16893c0975870b6b9e85149cbfce73770
• nx documentation --no-dte
• nx affected -t e2e-macos --parallel=1 --base=1572749f4a676d731ab761a045ff35426ab62a79 --head=0266dae16893c0975870b6b9e85149cbfce73770

---

Sent with 💌 from NxCloud.

[juristr]

Should we have this documented in other places too? Like in general for using PNPM with Angular?

[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.