👷 Adjus test and publish workflows

Test were missing the actual test run and publish would now dry run by default.
AndreMiras · Dec 4, 2024 · c956044 · c956044
1 parent 1a71b9f
commit c956044
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml
@@ -2,8 +2,7 @@ name: Publish
 
 on:
   push:
-    tags:
-      - "*"
+  pull_request:
 
 jobs:
   build:
@@ -17,6 +16,8 @@ jobs:
           registry-url: "https://registry.npmjs.org"
       - run: yarn install
       - run: yarn build
+      - run: npm publish --dry-run
       - run: npm publish
+        if: startsWith(github.ref, 'refs/tags/')
         env:
           NODE_AUTH_TOKEN: ${{ secrets.NODE_AUTH_TOKEN }}
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -19,4 +19,4 @@ jobs:
       - run: yarn install
       - run: yarn lint
       - run: yarn build
-      - run: npm publish --dry-run
+      - run: yarn test