Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(@angular/cli): during an update only use package manager force option with npm 7+ #23500

Merged
merged 1 commit into from Jul 6, 2022

Conversation

alan-agius4
Copy link
Collaborator

@alan-agius4 alan-agius4 commented Jul 4, 2022

In some cases previously we passed the force option to yarn which which lead to an installation failure.

Closes #23495

…tion with npm 7+

In some cases previously we passed the `force` option to yarn which which lead to an installation failure.

Closes angular#23495
@alan-agius4 alan-agius4 requested a review from clydin Jul 4, 2022
@alan-agius4 alan-agius4 added the target: patch label Jul 4, 2022
@alan-agius4 alan-agius4 marked this pull request as ready for review Jul 4, 2022
clydin
clydin approved these changes Jul 6, 2022
@alan-agius4 alan-agius4 added action: merge and removed action: review labels Jul 6, 2022
@alan-agius4 alan-agius4 merged commit cbccfd4 into angular:main Jul 6, 2022
13 checks passed
@alan-agius4 alan-agius4 deleted the yarn-install-force branch Jul 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge target: patch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants