Merge pull request #1433 from NullVoxPopuli/fix-ci--the-errorening

Make preflight error when suite-setup-util fails
embroider-build · May 18, 2023 · 18fd6a9 · 18fd6a9
2 parents 806723a + 82d20d1
commit 18fd6a9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -34,7 +34,9 @@ jobs:
         run: pnpm lint
       - id: set-matrix
         working-directory: test-packages/support
-        run: echo "matrix=$(node ./suite-setup-util.js --matrix)" >> $GITHUB_OUTPUT
+        run: |
+          matrix_json=$(node ./suite-setup-util --matrix)
+          echo "matrix=$matrix_json" >> $GITHUB_OUTPUT
 
   test:
     needs: ['preflight']

diff --git a/tests/scenarios/engines-test.ts b/tests/scenarios/engines-test.ts
@@ -117,7 +117,7 @@ engineScenarios
   });
 
 engineScenarios
-  .skip('release') // fails due to https://github.com/emberjs/ember.js/pull/20461
+  .skip('release-engines') // fails due to https://github.com/emberjs/ember.js/pull/20461
   .map('with-fastboot', app => {
     app.linkDependency('ember-cli-fastboot', { baseDir: __dirname });
     app.linkDependency('fastboot', { baseDir: __dirname });