split PR and merge workflows, upload only on merge (#103)

.github/workflows/build.yml → .github/workflows/build-common.yml

@@ -1,9 +1,13 @@
-name: "Build, check, test, release"
+name: "Build, check, test, release common"
 on:
-  push:
-    branches:
-      - 'agave-v2.0'
-  pull_request:
+  workflow_call:
+    inputs:
+      upload_artifacts:
+        required: true
+        type: boolean
+        default: false
+        description: "Upload artifacts to GitHub for consumption by fuzzcorp"
+
 
 jobs:
   test:
@@ -42,12 +46,14 @@ jobs:
           ./scripts/run_test_vectors.sh
 
       - uses: actions/upload-artifact@v4
+        if: ${{ inputs.upload_artifacts }}
         with:
           name: libsolfuzz-agave.so
           path: ./target/x86_64-unknown-linux-gnu/release/libsolfuzz_agave.so
 
       - name: Upload stubbed so
         uses: actions/upload-artifact@v4
+        if: ${{ inputs.upload_artifacts }}
         with:
           name: libsolfuzz-agave-stubbed.so
           path: ./target/x86_64-unknown-linux-gnu/release/libsolfuzz_agave_stubbed.so

.github/workflows/build-merge.yml

@@ -0,0 +1,12 @@
+name: "Build, check, test, release"
+on:
+  push:
+    branches:
+      - 'agave-v2.0'
+  pull_request:
+
+jobs:
+  build:
+    uses: ./.github/workflows/build-common.yml
+    with:
+      upload_artifacts: true

.github/workflows/build-pr.yml

@@ -0,0 +1,9 @@
+name: "Build, check, test"
+on:
+  pull_request:
+
+jobs:
+  build:
+    uses: ./.github/workflows/build-common.yml
+    with:
+      upload_artifacts: false