Skip to content

Commit

Permalink
Cleanup CI config
Browse files Browse the repository at this point in the history
- Update GitHub actions
- Remove duplicated entries in Go versions matrix:
  - '1.x' currently resolves to 'go1.19.4', just like '1.19'
  - `stable=false` option (which was removed in actions/setup-go@v3) enabled
    the installation of pre-release Go versions (now possible without
    that option), but it does not install "tip" version of Go
  - Jobs '1.x' with gostable=true and gostable=false were therefore
    duplicates of '1.19'
- Use default checkout path and default GOPATH
  • Loading branch information
olivierlemasle committed Jan 7, 2023
1 parent e202aff commit 9067c2d
Showing 1 changed file with 6 additions and 15 deletions.
21 changes: 6 additions & 15 deletions .github/workflows/ci.yml
Expand Up @@ -12,25 +12,17 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
go: [ '1.19', '1.18', '1.x' ]
gostable: [true]
include:
- go: '1.x'
gostable: false
go: [ '1.19', '1.18']
steps:
- uses: actions/checkout@v2
with:
path: src/k8s.io/kube-openapi/
- uses: actions/checkout@v3

- name: Set up Go
uses: actions/setup-go@v2
uses: actions/setup-go@v3
with:
go-version: "${{ matrix.go }}"
stable: ${{ matrix.gostable }}"

- name: Build
run: |
cd ${GITHUB_WORKSPACE}/src/k8s.io/kube-openapi/
go mod tidy && git diff --exit-code
go build ./cmd/... ./pkg/...
- name: Format
Expand All @@ -39,13 +31,12 @@ jobs:
if [[ -n "${diff}" ]]; then echo "${diff}"; exit 1; fi
- name: Test
run: |
cd ${GITHUB_WORKSPACE}/src/k8s.io/kube-openapi/
GOPATH=${GITHUB_WORKSPACE} go test ./cmd/... ./pkg/...
go test ./cmd/... ./pkg/...
- name: Run integration tests
run: |
cd ${GITHUB_WORKSPACE}/src/k8s.io/kube-openapi/test/integration
cd test/integration
go mod tidy && git diff --exit-code
GOPATH=${GITHUB_WORKSPACE} go test ./...
go test ./...
required:
# The name of the ci jobs above change based on the golang version.
# Use this as a stable required job that depends on the above jobs.
Expand Down

0 comments on commit 9067c2d

Please sign in to comment.