has-modern-meta: Correctly identify git changelings

PR-URL: #20390 Credit: @iarna Reviewed-By: @zkat
npm · Apr 20, 2018 · 2facb35 · 2facb35
1 parent e4ed976
commit 2facb35
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/lib/install/has-modern-meta.js b/lib/install/has-modern-meta.js
@@ -11,7 +11,10 @@ function isLink (child) {
 function hasModernMeta (child) {
   if (!child) return false
   const resolved = child.package._resolved && npa.resolve(moduleName(child), child.package._resolved)
-  return child.isTop || isLink(child) || child.fromBundle ||
-    child.package._inBundle || child.package._integrity ||
-    child.package._shasum || (resolved && resolved.type === 'git')
+  const version = npa.resolve(moduleName(child), child.package.version)
+  return child.isTop ||
+    isLink(child) ||
+    child.fromBundle || child.package._inBundle ||
+    child.package._integrity || child.package._shasum ||
+    (resolved && resolved.type === 'git') || (version && version.type === 'git')
 }