feat: allow test artifact upload to be optional

require it as an input
GeoNet · Oct 29, 2024 · 07ec667 · 07ec667
1 parent b10b9ae
commit 07ec667
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/.github/workflows/reusable-go-container-apps.yml b/.github/workflows/reusable-go-container-apps.yml
@@ -55,6 +55,10 @@ on:
         required: false
         description: |
           shell commands to setup the test environment
+      testUpload:
+        type: boolean
+        required: true
+        default: true
       golangciSetup:
         type: string
         required: false
@@ -146,6 +150,7 @@ jobs:
     with:
       setup: ${{ inputs.testSetup }}
       extraArgs: ${{ inputs.goTestExtraArgs }}
+      upload: ${{ inputs.testUpload }}
   go-vet:
     if: ${{ contains(fromJSON('["workflow_call", "workflow_dispatch", "push", "pull_request"]'), github.event_name) && startsWith(github.repository, 'GeoNet/') != false }}
     uses: GeoNet/Actions/.github/workflows/reusable-go-vet.yml@main

diff --git a/.github/workflows/reusable-go-test.yml b/.github/workflows/reusable-go-test.yml
@@ -14,7 +14,7 @@ on:
           extra args to pass `go test`
       upload:
         required: false
-        default: false
+        default: true
         type: boolean
         description: |
           set to true to push test coverage as an artifact