zrhoffman commented on a change in pull request #6026:
URL: https://github.com/apache/trafficcontrol/pull/6026#discussion_r674357741
##########
File path: .github/workflows/tm.integration.tests.yml
##########
@@ -48,7 +48,14 @@ jobs:
uses: actions/checkout@master
with:
fetch-depth: 1
+ - name: Check Go Version
+ run: echo "::set-output name=value::$(cat GO_VERSION)"
+ id: go-version
+ - name: Install Go
+ uses: actions/setup-go@v2
+ with:
+ go-version: ${{ steps.go-version.outputs.value }}
- name: Run integration tests
uses: ./.github/actions/tm-integration-tests
env:
- GOPATH: /github/workspace
+ GOPATH: /github/workspace
Review comment:
There should be a newline at the end of `tm.integration.tests.yml`.
##########
File path: .github/workflows/go.fmt.yml
##########
@@ -45,6 +45,13 @@ jobs:
steps:
- name: Checkout
uses: actions/checkout@master
+ - name: Check Go Version
+ run: echo "::set-output name=value::$(cat GO_VERSION)"
+ id: go-version
+ - name: Install Go
+ uses: actions/setup-go@v2
+ with:
+ go-version: ${{ steps.go-version.outputs.value }}
Review comment:
No need to set up Go on the GitHub Runner, since the *go-fmt* action
runs within Docker.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
