fix(@angular/core): mark zone.js as an optional peer dependency#61616
fix(@angular/core): mark zone.js as an optional peer dependency#61616alan-agius4 wants to merge 1 commit intoangular:mainfrom
zone.js as an optional peer dependency#61616Conversation
zone.js should be an optional peer dependencyzone.js as an optional peer dependency
c17c140 to
7f1c41a
Compare
`zone.js` is no longer mandatory.
7f1c41a to
30862f7
Compare
|
caretaker note: this change does not effect G3. |
|
Quick question: won't this break backwards compatibility with old projects that don't have ZoneJS in their dependency list, since Angular 2+ would automatically grab it? |
|
npm (v7+) and pnpm (v7+) support auto-installing peer dependencies, including those explicitly marked as optional so that should not be a problem. Although in general, it would be recommended to have a stricter dependency policy and avoid auto-installing peer dependency and have them explicitly listed. |
`zone.js` is no longer mandatory. PR Close #61616
|
This PR was merged into the repository by commit 229cdde. The changes were merged into the following branches: main, 20.0.x |
|
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
zone.jsis no longer mandatory.