upgrade npm to 3.9.0

[iarna]

Checklist

• tests and code linting passes
• a test and/or benchmark is included
• documentation is changed or added
• the commit message follows commit guidelines

Affected core subsystem(s)

• deps

Description of change

The biggest thing in this release is that it fixes the tests on Windows.

There are a couple of Windows related bug fixes that were exposed by fixing the tests on Windows. Notably:

• The use of npm explore to run a command from the directory of a dependency didn't work correctly on Windows.
• npm init and others that inferred the module name from the path failed with scoped modules on Windows.
• Removing globally installed bins for scoped modules previously failed on Windows. (And was incorrect, but working by-accident elsewhere.)
• Correct npm cache ls to output real filenames– previously it sometimes mistakenly double the package name in the path, resulting paths that don't exist.
• npm now warns if you paste in a command line that uses non-ascii dashes (like emdashes or endashes), as this is almost certainly not intended.

And one new feature:

• If we got a cache-hit for module metadata but the version requested does not exist in the metadata, we then explicitly ask the registry for updated metadata. This removes one of the primary pain points of using a very high --cache-min.

Changelogs

• v3.9.0

Reviewers

r: @zkat
r: @thealphanerd
r: @Fishrock123

[MylesBorins]

ci: https://ci.nodejs.org/job/node-test-pull-request/2660/

[jasnell]

@iarna ... I'm not too familiar with coveralls but I want to verify... was including the COVERALLS_REPO_TOKEN here and the slack incoming_webhook secure token intentional?

[iarna]

@jasnell We've included the slack one in Travis throughout. They're all public in the npm repo, so there's little harm in them being here.

[Fishrock123]

Fwiw:

Applying: deps: upgrade npm to 3.9.0
/Users/Jeremiah/Documents/node/.git/rebase-apply/patch:1098: trailing whitespace.
The `--ignore-scripts` argument will cause npm to not execute any 
/Users/Jeremiah/Documents/node/.git/rebase-apply/patch:1111: trailing whitespace.
3. Developer, works on packages they are given access to.  
/Users/Jeremiah/Documents/node/.git/rebase-apply/patch:1120: trailing whitespace.
* Grant access:  
/Users/Jeremiah/Documents/node/.git/rebase-apply/patch:1133: trailing whitespace.
default will only publish internally. 
/Users/Jeremiah/Documents/node/.git/rebase-apply/patch:1366: trailing whitespace.
<p>The <code>--ignore-scripts</code> argument will cause npm to not execute any 
warning: squelched 1133 whitespace errors
warning: 1035 lines applied after fixing whitespace errors.

That's an awful lot lol

[MylesBorins]

local tests pass make test-npm with this patch applied to master

[Fishrock123]

Tests also pass for me. @iarna Do the whitespace squelches concern you at all? If not, I'll go ahead and land this.

[iarna]

Let me fix the whitespace nonsense. I thought I'd done that, but I think I crossed wires with the 2.x branch.

[iarna]

Whitespace fixed.

[chicoxyzzy]

notice npm/npm#12647

[Fishrock123]

@iarna Is there anything critical in this release? The above issue sounds like we should hold out for 3.9.1 or similar?

[MylesBorins]

closing in favor of #7139