Merge pull request #576 from mkszepp/remove-prepublish

Remove `prepublishOnly` in favor of `prepack` script to fix randomly test failing
adopted-ember-addons · Aug 4, 2024 · 704af2d · 704af2d
2 parents 64ab29c + 7272f11
commit 704af2d
Show file tree

Hide file tree

Showing 4 changed files with 896 additions and 392 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -79,5 +79,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: wyvox/action-setup-pnpm@v3
       - name: Run Tests
-        run: ./node_modules/.bin/ember try:one ${{ matrix.try-scenario }} --skip-cleanup
+        run: |
+          pnpm _syncPnpm
+          ./node_modules/.bin/ember try:one ${{ matrix.try-scenario }} --skip-cleanup
         working-directory: test-app
diff --git a/addon/package.json b/addon/package.json
@@ -27,7 +27,7 @@
   ],
   "scripts": {
     "build": "rollup --config",
-    "prepublishOnly": "rollup --config",
+    "prepack": "rollup --config",
     "lint": "npm-run-all --aggregate-output --continue-on-error --parallel \"lint:!(fix)\"",
     "lint:js": "eslint . --cache",
     "lint:js:fix": "eslint . --fix",