Skip to content

test(@angular/cli): verify package group locking under peer updates#33549

Merged
clydin merged 1 commit into
angular:mainfrom
clydin:test-package-group-update-peer
Jul 10, 2026
Merged

test(@angular/cli): verify package group locking under peer updates#33549
clydin merged 1 commit into
angular:mainfrom
clydin:test-package-group-update-peer

Conversation

@clydin

@clydin clydin commented Jul 10, 2026

Copy link
Copy Markdown
Member

Asserts that the update resolver loops correctly prioritize package group expansions before evaluating peer dependencies.

This prevents regression of issue #22654, where specifying a package with a wide pre-release peer range (like @angular/cdk) before a package defining a group (like @angular/core) would resolve group dependencies (like @angular/common) to next-major pre-releases instead of locking them to target group versions.

Asserts that the update resolver loops correctly prioritize package group expansions before evaluating peer dependencies.

This prevents regression of issue angular#22654, where specifying a package with a wide pre-release peer range (like @angular/cdk) before a package defining a group (like @angular/core) would resolve group dependencies (like @angular/common) to next-major pre-releases instead of locking them to target group versions.
@clydin clydin added the target: minor This PR is targeted for the next minor release label Jul 10, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds mock package definitions and a new unit test to update-resolver_spec.ts to verify that packageGroup versions are correctly maintained when updating packages alongside peer dependencies. There are no review comments, so no additional feedback is provided.

@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Jul 10, 2026
@clydin clydin requested a review from alan-agius4 July 10, 2026 18:09
@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 Jul 10, 2026
@clydin clydin merged commit e5fff87 into angular:main Jul 10, 2026
43 checks passed
@clydin

clydin commented Jul 10, 2026

Copy link
Copy Markdown
Member Author

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

@clydin clydin deleted the test-package-group-update-peer branch July 10, 2026 20:22
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