Update for npm, remove yarn

emberjs · Jan 14, 2025 · 06a19fc · 06a19fc
1 parent 2662cde
commit 06a19fc
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/.github/workflows/compat-tests.yml b/.github/workflows/compat-tests.yml
@@ -90,7 +90,10 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        packageManager: [npm, yarn, pnpm]
+        packageManager:
+        - npm
+        # - yarn # yarn@1 has not been reliable, if yarn@4 were easy to setup, we could test against that
+        - pnpm
         kind:
           - false # Test vanilla pnpm -r
           - alpha # requires unstable-preview-types folder for each package

diff --git a/scripts/test-package-manager.mjs b/scripts/test-package-manager.mjs
@@ -261,7 +261,13 @@ async function runNoThrow(cwd, cmd) {
 
 async function install(packageManager, cwd) {
   // All package managers have an install command
-  await $({ preferLocal: true, shell: true, cwd, stdio: 'inherit' })`${packageManager} install`;
+  switch (packageManager) {
+    case 'npm':
+      // npm complains about tgz files in the version specifier part of package.json
+      await $({ preferLocal: true, shell: true, cwd, stdio: 'inherit' })`${packageManager} install --force`;
+    default:
+      await $({ preferLocal: true, shell: true, cwd, stdio: 'inherit' })`${packageManager} install`;
+  }
 }
 
 async function typecheck(packageManager, cwd) {