deps: upgrade npm to 6.1.0

PR-URL: https://github.com/nodejs/node/pull/20190
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>

1 files changed, 6 insertions, 2 deletions

diff --git a/deps/npm/node_modules/update-notifier/index.js b/deps/npm/node_modules/update-notifier/index.js
index 58f2459036..38ff01e295 100644
--- a/deps/npm/node_modules/update-notifier/index.js
+++ b/deps/npm/node_modules/update-notifier/index.js
@@ -12,6 +12,7 @@ const isNpm = importLazy('is-npm');
 const isInstalledGlobally = importLazy('is-installed-globally');
 const boxen = importLazy('boxen');
 const xdgBasedir = importLazy('xdg-basedir');
+const isCi = importLazy('is-ci');
 const ONE_DAY = 1000 * 60 * 60 * 24;
 
 class UpdateNotifier {
@@ -37,7 +38,9 @@ class UpdateNotifier {
 		this.hasCallback = typeof options.callback === 'function';
 		this.callback = options.callback || (() => {});
 		this.disabled = 'NO_UPDATE_NOTIFIER' in process.env ||
-			process.argv.indexOf('--no-update-notifier') !== -1;
+			process.argv.indexOf('--no-update-notifier') !== -1 ||
+			isCi();
+		this.shouldNotifyInNpmScript = options.shouldNotifyInNpmScript;
 
 		if (!this.disabled && !this.hasCallback) {
 			try {
@@ -106,7 +109,8 @@ class UpdateNotifier {
 		});
 	}
 	notify(opts) {
-		if (!process.stdout.isTTY || isNpm() || !this.update) {
+		const suppressForNpm = !this.shouldNotifyInNpmScript && isNpm();
+		if (!process.stdout.isTTY || suppressForNpm || !this.update) {
 			return this;
 		}