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: scope config #1758

Closed

Conversation

ruyadorno
Copy link
Contributor

The flatOptions.scope option should not default to projectScope since that's causing projects that have a defined projectScope corresponding to a configured scoped registry to try and download ALL packages (including the ones that should be downloaded from the public registry) to be fetch from that scoped registry url (as if npm install --scope="@myscope").

This PR fixes it by reinstating a regular flatOptions.scope config.

fix #1654

The `flatOptions.scope` option should not default to `projectScope`
since that's causing projects that have a defined `projectScope`
corresponding to configured scoped registries to try and download ALL
packages (including the ones that should be downloaded from the public
registry) to be fetch from that scoped registry url.

fix npm#1654
@ruyadorno ruyadorno requested a review from a team as a code owner September 3, 2020 00:48
@ruyadorno ruyadorno added Release 7.x work is associated with a specific npm 7 release Bug thing that needs fixing labels Sep 3, 2020
isaacs pushed a commit that referenced this pull request Sep 4, 2020
The `flatOptions.scope` option should not default to `projectScope`
since that's causing projects that have a defined `projectScope`
corresponding to configured scoped registries to try and download ALL
packages (including the ones that should be downloaded from the public
registry) to be fetch from that scoped registry url.

fix #1654

PR-URL: #1758
Credit: @ruyadorno
Close: #1758
Reviewed-by: @isaacs
@ruyadorno ruyadorno closed this Sep 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug thing that needs fixing Release 7.x work is associated with a specific npm 7 release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant