Skip to content

fix(@angular/cli): conditionally quote package names when adding dependencies based on host requirements#32704

Merged
alan-agius4 merged 1 commit intoangular:mainfrom
alan-agius4:specifier
Mar 10, 2026
Merged

fix(@angular/cli): conditionally quote package names when adding dependencies based on host requirements#32704
alan-agius4 merged 1 commit intoangular:mainfrom
alan-agius4:specifier

Conversation

@alan-agius4
Copy link
Collaborator

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.

…ndencies based on host requirements

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.
@alan-agius4 alan-agius4 requested a review from clydin March 5, 2026 13:07
@alan-agius4 alan-agius4 added the target: patch This PR is targeted for the next patch release label Mar 5, 2026
@alan-agius4 alan-agius4 added the action: review The PR is still awaiting reviews from at least one requested reviewer label Mar 5, 2026
gemini-code-assist[bot]

This comment was marked as off-topic.

@alan-agius4 alan-agius4 closed this Mar 5, 2026
@alan-agius4 alan-agius4 reopened this Mar 5, 2026
@alan-agius4 alan-agius4 requested review from clydin and removed request for clydin March 9, 2026 15:10
@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 Mar 10, 2026
@alan-agius4 alan-agius4 merged commit 598a690 into angular:main Mar 10, 2026
69 of 71 checks passed
@alan-agius4 alan-agius4 deleted the specifier branch March 10, 2026 10:18
@alan-agius4
Copy link
Collaborator Author

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants