repo: Install npm dependencies with the --no-save flag

[timmywil]

This fixes jquery/jquery#3947

• Also, I'm not sure -q does anything anymore. It's not documented in the npm install docs.

[frank-dspeed]

LGTM

[Silic0nS0ldier]

-q sets logging to warnings or greater. Only documented at their repo though. Still works as of npm 5.6.0.

[markelog]

It is documented, see – https://docs.npmjs.com/misc/config

[markelog]

It seems -q is there to remove needless output?

[timmywil]

Maybe our definition of needless differs, but I don't see any needless output. This may differ from older versions because of the new loading animation.

[markelog]

That’s a good guess about animation! I’m fine if you want to land it as is, just wanted to point it out :) Regards, Oleg

…

On 22 Jan 2018, at 19:22, Timmy Willison ***@***.***> wrote: Maybe our definition of needless differs, but I don't see any needless output. This may differ from older versions because of the new loading animation. — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

[Silic0nS0ldier]

@markelog Huh. So it is. Was expecting it to be under cli. Thanks for the pointer.

The only thing -q seems to suppress under normal use is the package-lock.json creation notice, which won't happen anyway due to --no-save. It likely also stopped the dependency tree print out in older versions.

Huh. As of v5 of NPM, the --save switch is the default. Chances are someone updated their version of npm (either directly or via a node update). Another change like this is unlikely to hit again soon (if ever), but it might be worth locking down the npm version if possible.