Respect ignore engines flag.

Fixes #638
yarnpkg · Oct 11, 2016 · 20836da · 20836da
1 parent 131bc3a
commit 20836da
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/cli/commands/install.js b/src/cli/commands/install.js
@@ -148,7 +148,7 @@ export class Install {
 
     this.resolver = new PackageResolver(config, lockfile);
     this.fetcher = new PackageFetcher(config, this.resolver);
-    this.compatibility = new PackageCompatibility(config, this.resolver);
+    this.compatibility = new PackageCompatibility(config, this.resolver, this.flags.ignoreEngines);
     this.linker = new PackageLinker(config, this.resolver, this.flags.ignoreOptional);
     this.scripts = new PackageInstallScripts(config, this.resolver, this.flags.force);
   }

diff --git a/src/package-compatibility.js b/src/package-compatibility.js
@@ -94,10 +94,11 @@ export function testEngine(name: string, range: string, versions: Versions, loos
 }
 
 export default class PackageCompatibility {
-  constructor(config: Config, resolver: PackageResolver) {
+  constructor(config: Config, resolver: PackageResolver, ignoreEngines: boolean) {
     this.reporter = config.reporter;
     this.resolver = resolver;
     this.config = config;
+    this.ignoreEngines = ignoreEngines;
   }
 
   resolver: PackageResolver;
@@ -148,7 +149,7 @@ export default class PackageCompatibility {
       }
     }
 
-    if (!this.config.ignoreEngines && typeof info.engines === 'object') {
+    if (!this.ignoreEngines && typeof info.engines === 'object') {
       for (const entry of entries(info.engines)) {
         let name = entry[0];
         const range = entry[1];