Refactor upgrades #9929

Closed
wants to merge 4 commits into
from

Projects

None yet

2 participants

@iarna
Member
iarna commented Oct 10, 2015

By which I mean, when npm decides it needs to upgrade an existing module, this changes how it does that. This applies to both npm install foo@newversion and updating a package.json and running npm install. In either case, it does a better job of cleaning out the older deps, which means that the resulting tree is flatter!

Fixes: #9520

@iarna iarna added the in progress label Oct 10, 2015
@iarna iarna changed the title from Refactor a upgrades to Refactor upgrades Oct 12, 2015
@iarna iarna added this to the next-3 milestone Oct 12, 2015
@othiym23
Contributor

See also #9633.

@iarna
Member
iarna commented Oct 15, 2015

I don't feel like this got enough of a review, so I'm pushing it out a week. Plus CI was being weird with it and that concerns me.

@iarna iarna modified the milestone: next-next-3, next-3 Oct 15, 2015
@iarna iarna added a commit that referenced this pull request Oct 22, 2015
@iarna iarna deps: Refactor module matchers into stand alone functions
PR-URL: #9929
e47eed2
@iarna iarna added a commit that referenced this pull request Oct 22, 2015
@iarna iarna deps: Move adding to a set of module names to a function
PR-URL: #9929
5799642
@iarna iarna added a commit that referenced this pull request Oct 22, 2015
@iarna iarna deps: When we replace/upgrade a dep, remove all its deps too
PR-URL: #9929
971fd47
@iarna
Member
iarna commented Oct 23, 2015

This landed in 3.3.10

@iarna iarna closed this Oct 23, 2015
@iarna iarna removed the in progress label Oct 23, 2015
@iarna iarna deleted the iarna/refactor-upgrades branch Oct 28, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment