Skip to content

Conversation

@clydin
Copy link
Member

@clydin clydin commented Dec 29, 2025

This commit adds support for ignoring peer dependency warnings and errors during package installation. This is useful for commands like ng update where transient peer dependency conflicts may occur.

  • Added ignorePeerDependenciesFlag to the PackageManagerDescriptor interface.
  • Implemented the flag for npm (--force) and pnpm (--strict-peer-dependencies=false).
  • Updated PackageManager.install to accept and apply the ignorePeerDependencies option.

@clydin clydin added the target: minor This PR is targeted for the next minor release label Dec 29, 2025
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Dec 29, 2025
clydin added 2 commits January 5, 2026 11:12
…manager

This commit adds support for ignoring peer dependency warnings and errors during package installation. This is useful for commands like `ng update` where transient peer dependency conflicts may occur.

- Added `ignorePeerDependenciesFlag` to the `PackageManagerDescriptor` interface.
- Implemented the flag for `npm` (`--force`) and `pnpm` (`--strict-peer-dependencies=false`).
- Updated `PackageManager.install` to accept and apply the `ignorePeerDependencies` option.
…ommand

This commit updates the `ng update` command to utilize the `ignorePeerDependencies` abstraction option when installing packages. Previously, the command forced the installation (`--force`) when NPM 7+ was detected to workaround peer dependency issues. Now, it uses the more specific `ignorePeerDependencies` option (mapping to `--force` for NPM).
@clydin clydin force-pushed the package-manager/ignore-peer-deps branch from 944caf2 to 7e625c3 Compare January 5, 2026 16:12
@clydin clydin requested a review from alan-agius4 January 5, 2026 16:30
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jan 6, 2026
@clydin clydin merged commit eb3a850 into angular:main Jan 6, 2026
35 checks passed
@clydin
Copy link
Member Author

clydin commented Jan 6, 2026

This PR was merged into the repository. The changes were merged into the following branches:

@clydin clydin deleted the package-manager/ignore-peer-deps branch January 6, 2026 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: @angular/cli target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants