Skip to content

fix(@angular/cli): quote complex range specifiers in package manager#32540

Merged
clydin merged 1 commit intoangular:mainfrom
clydin:pm/complex-ranges
Feb 24, 2026
Merged

fix(@angular/cli): quote complex range specifiers in package manager#32540
clydin merged 1 commit intoangular:mainfrom
clydin:pm/complex-ranges

Conversation

@clydin
Copy link
Member

@clydin clydin commented Feb 23, 2026

Complex range specifiers that include shell special characters (e.g., '>', '<') can be misinterpreted when not quoted. This change ensures that version ranges are always enclosed in quotes to prevent such issues.

@clydin clydin added the target: patch This PR is targeted for the next patch release label Feb 23, 2026
@clydin clydin added target: minor This PR is targeted for the next minor release target: rc This PR is targeted for the next release-candidate and removed target: patch This PR is targeted for the next patch release target: minor This PR is targeted for the next minor release labels Feb 23, 2026
@clydin clydin force-pushed the pm/complex-ranges branch 2 times, most recently from 79821fe to 5816dc2 Compare February 23, 2026 17:12
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Feb 23, 2026
@clydin clydin requested a review from alan-agius4 February 23, 2026 17:43
Complex range specifiers that include shell special characters (e.g., '>', '<') can be misinterpreted when not quoted. This change ensures that version ranges are enclosed in quotes when needed to prevent such issues.

A test case has been added to verify that complex specifiers are handled correctly.
@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 Feb 24, 2026
@clydin clydin merged commit 85e3ce2 into angular:main Feb 24, 2026
60 of 63 checks passed
@clydin
Copy link
Member Author

clydin commented Feb 24, 2026

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

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: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants