ci: fix unit test artifact name collisions [v8] (#540)

In a previous PR, I had refactored the go-setup step out of the unit
tests.

Unfortunately, this had the side-effect of the unit test action no
longer knowing what Go version it was using (since it used to be passed
in as a parameter.) This meant that when run in a matrix build, the test
result uploads would clash and fail to upload.

The fix is to check the current Go version using `go version` inside the
action.

.github/actions/unit-tests/action.yml

@@ -1,9 +1,6 @@
 name: Unit Tests
 description: "Runs Relay's unit tests + linters and optionally gathers coverage."
 inputs:
-  go-version:
-    description: 'Go version to use for build & test.'
-    required: true
   lint:
     description: 'Whether to run linters.'
     required: false
@@ -16,6 +13,11 @@ inputs:
 runs:
   using: composite
   steps:
+    - name: Get Go version
+      id: go-version
+      shell: bash
+      run: |
+        echo "version=$(go version | awk '{print $3}')" >> $GITHUB_OUTPUT
     - name: Lint
       if: inputs.lint == 'true'
       shell: bash
@@ -43,12 +45,12 @@ runs:
       if: steps.process-test.outcome == 'success'
       uses: actions/upload-artifact@v4
       with:
-        name: Test-result-go${{ inputs.go-version }}
+        name: Test-result-${{ steps.go-version.outputs.version }}
         path: junit_report.xml
 
     - name: Upload coverage results
       if: steps.test-coverage.outcome == 'success'
       uses: actions/upload-artifact@v4
       with:
-        name: Coverage-result-go${{ inputs.go-version }}
+        name: Coverage-result-${{ steps.go-version.outputs.version }}
         path: build/coverage*

.github/workflows/manual-publish.yml

@@ -40,7 +40,6 @@ jobs:
       - name: Publish Package
         uses: ./.github/actions/publish
         with:
-          go-version: ${{ needs.go-versions.outputs.latest }}
           token: ${{ secrets.GITHUB_TOKEN }}
           dry-run: ${{ inputs.dry_run }}
           tag: ${{ inputs.tag }}