ci: refactor (#43)

sripwoud · Jul 12, 2024 · 5896624 · 5896624
1 parent 64c0aa5
commit 5896624
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 65 deletions.
diff --git a/.github/workflows/main.yaml b/.github/workflows/main.yaml
@@ -7,6 +7,8 @@ on:
   pull_request:
     branches: [main]
 
+# no need to bother caching bun deps
+# https://github.com/oven-sh/setup-bun/issues/14#issuecomment-1714116221
 jobs:
   changed-files:
     runs-on: ubuntu-latest
@@ -22,7 +24,7 @@ jobs:
             package.json
             tsconfig.json
 
-  test:
+  _test:
     needs: changed-files
     if: needs.changed-files.outputs.any_ts_changed == 'true'
     runs-on: ubuntu-latest
@@ -37,6 +39,16 @@ jobs:
           format: lcov
           file: lcov.info
 
+  test:
+    needs: _test
+    # workaround for https://github.com/orgs/community/discussions/13690
+    # https://stackoverflow.com/a/77066140/9771158
+    if: ${{ !(failure() || cancelled()) }}
+    runs-on: ubuntu-latest
+    steps:
+      - name: Tests OK (passed or skipped)
+        run: true
+
   validate:
     runs-on: ubuntu-latest
     steps:

diff --git a/.github/workflows/validate.yaml b/.github/workflows/validate.yaml