Removes warnings from package manifest

ruyadorno · isaacs · commit 3547d99892a3 · 2019-10-07T16:29:14.000-07:00
diff --git a/lib/install/validate-tree.js b/lib/install/validate-tree.js
@@ -39,8 +39,9 @@ function checkErrors (mod, idealTree, next) {
 
 function checkWarnings (mod, idealTree, next) {
   const warnings = mod.package.warnings
-  if (warnings && (mod.parent || path.resolve(npm.globalDir, '..'))) {
+  if (warnings && (mod.parent || path.resolve(npm.globalDir, '..') !== mod.path)) {
     warnings.forEach(warn => idealTree.warnings.push(warn))
+    delete mod.package.warnings
   }
   next()
 }

Original file line number	Diff line number	Diff line change
`@@ -39,8 +39,9 @@ function checkErrors (mod, idealTree, next) {`
`39`	`39`
`40`	`40`	`function checkWarnings (mod, idealTree, next) {`
`41`	`41`	`const warnings = mod.package.warnings`
`42`		`- if (warnings && (mod.parent \|\| path.resolve(npm.globalDir, '..'))) {`
	`42`	`+ if (warnings && (mod.parent \|\| path.resolve(npm.globalDir, '..') !== mod.path)) {`
`43`	`43`	`warnings.forEach(warn => idealTree.warnings.push(warn))`
	`44`	`+ delete mod.package.warnings`
`44`	`45`	`}`
`45`	`46`	`next()`
`46`	`47`	`}`