Skip to content

refactor(@angular/cli): implement lazy validation for package manager#32417

Merged
clydin merged 1 commit intoangular:mainfrom
clydin:pm/lazy-validation
Feb 4, 2026
Merged

refactor(@angular/cli): implement lazy validation for package manager#32417
clydin merged 1 commit intoangular:mainfrom
clydin:pm/lazy-validation

Conversation

@clydin
Copy link
Member

@clydin clydin commented Feb 3, 2026

This changes the package manager initialization to only throw errors when the binary is actually required for an operation. This allows CLI commands that do not depend on the package manager binary to function even if the configured package manager is missing.

@clydin clydin added the target: minor This PR is targeted for the next minor release label Feb 3, 2026
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Feb 3, 2026
@clydin clydin requested a review from alan-agius4 February 3, 2026 21:10
@clydin clydin force-pushed the pm/lazy-validation branch from 9319949 to 438bcdd Compare February 3, 2026 21:46
Copy link
Collaborator

@alan-agius4 alan-agius4 left a comment

Choose a reason for hiding this comment

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

LGTM, just a NIT

@alan-agius4 alan-agius4 removed the action: review The PR is still awaiting reviews from at least one requested reviewer label Feb 4, 2026
@clydin clydin force-pushed the pm/lazy-validation branch from 438bcdd to 7a8f650 Compare February 4, 2026 14:04
@clydin clydin added the action: merge The PR is ready for merge by the caretaker label Feb 4, 2026
This changes the package manager initialization to only throw errors when the binary is actually required for an operation. This allows CLI commands that do not depend on the package manager binary to function even if the configured package manager is missing.
@clydin clydin force-pushed the pm/lazy-validation branch from 7a8f650 to 530f6a9 Compare February 4, 2026 14:21
@clydin clydin merged commit fe4c310 into angular:main Feb 4, 2026
34 checks passed
@clydin
Copy link
Member Author

clydin commented Feb 4, 2026

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

@clydin clydin deleted the pm/lazy-validation branch February 4, 2026 14:36
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