Merge pull request #994 from ember-cli/ignore-scripts-by-default

Author: kategengler

lib/dependency-manager-adapters/pnpm.js

@@ -122,6 +122,10 @@ module.exports = CoreObject.extend({
       if (!mgrOptions.includes('--no-lockfile')) {
         mgrOptions.push('--no-lockfile');
       }
+
+      if (!mgrOptions.includes('--ignore-scripts')) {
+        mgrOptions.push('--ignore-scripts');
+      }
     }
 
     // Note: We are explicitly *not* using `--no-lockfile` here, so that we

test/dependency-manager-adapters/pnpm-adapter-test.js

@@ -121,7 +121,7 @@ describe('pnpm Adapter', () => {
       let stubbedRun = generateMockRun(
         [
           {
-            command: 'pnpm install --no-lockfile',
+            command: 'pnpm install --no-lockfile --ignore-scripts',
             async callback(command, args, opts) {
               runCount++;
               expect(opts).to.have.property('cwd', tmpdir);
@@ -226,7 +226,7 @@ describe('pnpm Adapter', () => {
       let stubbedRun = generateMockRun(
         [
           {
-            command: 'pnpm install --no-lockfile',
+            command: 'pnpm install --no-lockfile --ignore-scripts',
             async callback(command, args, opts) {
               runCount++;
               expect(opts).to.have.property('cwd', tmpdir);