Test against embroider (#479)

* Also test against strictest embroider * Add missing dependencies * Add maybeEmbroider to the other test-apps * more default * nevermind about the ember-concurrency test-app, it seems incompatible
emberjs · May 6, 2024 · 120b599 · 120b599
1 parent 3e00691
commit 120b599
Show file tree

Hide file tree

Showing 6 changed files with 922 additions and 356 deletions.
diff --git a/.github/workflows/ci-build.yml b/.github/workflows/ci-build.yml
@@ -13,26 +13,42 @@ on:
 jobs:
   test:
     timeout-minutes: 10
-    name: Tests
+    name: "Tests: ${{ matrix.EMBROIDER && 'embroider' || 'classic' }}"
     runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        EMBROIDER:
+          - '' # classic
+          - 'true'
 
     steps:
       - uses: actions/checkout@v4
       - uses: wyvox/action-setup-pnpm@v3
       - run: pnpm lint
       - run: pnpm test:ember
         working-directory: test-apps/base-tests
+        env: 
+          EMBROIDER: ${{ matrix.EMBROIDER }}
 
   floating-dependencies:
     timeout-minutes: 10
-    name: "Floating Dependencies"
+    name: "Floating Dependencies: ${{ matrix.EMBROIDER && 'embroider' || 'classic' }}"
     runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        EMBROIDER:
+          - '' # classic
+          - 'true'
 
     steps:
       - uses: actions/checkout@v4
       - uses: wyvox/action-setup-pnpm@v3
       - run: pnpm test:ember
         working-directory: test-apps/base-tests
+        env: 
+          EMBROIDER: ${{ matrix.EMBROIDER }}
 
   typecheck:
     name: '${{ matrix.typescript-scenario }}'