/meteor

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 accidental stripping of @scoped npm packages. #7609

Merged
merged 1 commit into from Aug 8, 2016

Conversation

Reviewers
No reviews
Assignees

@benjamn benjamn

Labels
None yet
Projects
None yet
Milestone
  Release 1.4.1
1 participant
@benjamn
@benjamn
Member

benjamn commented Aug 8, 2016

The previous code did not accommodate the possibility that meteorNpm.getProdPackageNames might return a package name that contained one or more / characters.

Fixes #7579.
@benjamn
Fix accidental stripping of @Scoped npm packages. 
The previous code did not accommodate the possibility that
meteorNpm.getProdPackageNames might return a package name that contained
one or more `/` characters.

Fixes #7579.
Loading status checks…
ff9c2c0

@benjamn benjamn added this to the Release 1.4.1 milestone Aug 8, 2016

@benjamn benjamn self-assigned this Aug 8, 2016

@benjamn benjamn merged commit a912eed into devel Aug 8, 2016
4 checks passed

4 checks passed

CLA Author has signed the Meteor CLA.
Details
ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@benjamn benjamn referenced this pull request Aug 12, 2016

Closed

NPM Modules under organizations are being bundled into modules.js #7640

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment