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

don't shorten node_modules to ~ #4144

Merged
merged 2 commits into from May 17, 2017
Merged

don't shorten node_modules to ~ #4144

merged 2 commits into from May 17, 2017

Conversation

sokra
Copy link
Member

@sokra sokra commented Jan 31, 2017

What kind of change does this PR introduce?
change, fix

Did you add tests for your changes?
not yet

If relevant, link to documentation update:
n/a

Summary
don't shorten node_modules to ~
it's confusing

Does this PR introduce a breaking change?
yes

Other information

@sokra sokra added this to the webpack 3 milestone Feb 1, 2017
@TheLarkInn
Copy link
Member

Is there an issue we can tie to this? Just to track.

@nojvek
Copy link

nojvek commented Mar 16, 2017

@sokra Thank you so much for working on this.

@webpack-bot
Copy link
Contributor

Thank you for your pull request! The most important CI builds succeeded, we’ll review the pull request soon.

@sokra sokra changed the base branch from master to next May 16, 2017 10:04
@sokra sokra closed this May 16, 2017
@sokra sokra reopened this May 16, 2017
@sokra sokra closed this May 16, 2017
@sokra sokra reopened this May 16, 2017
@sokra sokra merged commit 2fa2f61 into next May 17, 2017
@sokra sokra deleted the fix/node_modules_mangle branch May 17, 2017 06:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants