mirror of
https://github.com/ethereum/go-ethereum.git
synced 2026-07-15 17:30:44 +00:00
chore(ci): add go catch-all job (#140)
## Why this should be merged New jobs are easily forgotten in the GitHub rules so we will only gate on `go` and have all others added as dependencies. ## How this works Same as #139 ## How this was tested Inspection of CI run: 
This commit is contained in:
parent
bc42e5f29c
commit
f7a3a4f548
2 changed files with 11 additions and 12 deletions
18
.github/workflows/go.yml
vendored
18
.github/workflows/go.yml
vendored
|
|
@ -12,6 +12,14 @@ concurrency:
|
||||||
cancel-in-progress: ${{ github.ref != 'refs/heads/main' }}
|
cancel-in-progress: ${{ github.ref != 'refs/heads/main' }}
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
|
# If adding a new job, add it to the `needs` list of the `go` job as this is
|
||||||
|
# what gates PRs.
|
||||||
|
go:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
needs: [go_test_short, go_test_tooling, go_generate, go_tidy]
|
||||||
|
steps:
|
||||||
|
- run: echo "Dependencies successful"
|
||||||
|
|
||||||
go_test_short:
|
go_test_short:
|
||||||
env:
|
env:
|
||||||
FLAKY_REGEX: "ava-labs/libevm/(triedb/pathdb|eth|eth/tracers/js|eth/tracers/logger|accounts/abi/bind|accounts/keystore|eth/downloader|miner|ethclient|ethclient/gethclient|eth/catalyst)$"
|
FLAKY_REGEX: "ava-labs/libevm/(triedb/pathdb|eth|eth/tracers/js|eth/tracers/logger|accounts/abi/bind|accounts/keystore|eth/downloader|miner|ethclient|ethclient/gethclient|eth/catalyst)$"
|
||||||
|
|
@ -62,7 +70,7 @@ jobs:
|
||||||
- name: git diff
|
- name: git diff
|
||||||
run: git diff --exit-code
|
run: git diff --exit-code
|
||||||
|
|
||||||
go_tidy_matrix:
|
go_tidy:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
strategy:
|
strategy:
|
||||||
matrix:
|
matrix:
|
||||||
|
|
@ -78,11 +86,3 @@ jobs:
|
||||||
go-version-file: "${{ matrix.dir }}/go.mod"
|
go-version-file: "${{ matrix.dir }}/go.mod"
|
||||||
- run: go mod tidy
|
- run: go mod tidy
|
||||||
- run: git diff --exit-code
|
- run: git diff --exit-code
|
||||||
|
|
||||||
go_tidy:
|
|
||||||
# Each matrix instance runs as a different name, which doesn't play nicely
|
|
||||||
# with branch-protection rules. We instead require this job.
|
|
||||||
needs: go_tidy_matrix
|
|
||||||
runs-on: ubuntu-latest
|
|
||||||
steps:
|
|
||||||
- run: echo "Dependencies successful"
|
|
||||||
|
|
|
||||||
5
.github/workflows/lint.yml
vendored
5
.github/workflows/lint.yml
vendored
|
|
@ -1,8 +1,5 @@
|
||||||
name: lint
|
name: lint
|
||||||
|
|
||||||
# If adding a new linter: (a) create a new job; and (b) add it to the `needs`
|
|
||||||
# list of the `lint` job as this is what gates PRs.
|
|
||||||
|
|
||||||
on:
|
on:
|
||||||
push:
|
push:
|
||||||
branches: [main, "release/**"]
|
branches: [main, "release/**"]
|
||||||
|
|
@ -21,6 +18,8 @@ concurrency:
|
||||||
cancel-in-progress: ${{ github.ref != 'refs/heads/main' }}
|
cancel-in-progress: ${{ github.ref != 'refs/heads/main' }}
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
|
# If adding a new linter: (a) create a new job; and (b) add it to the `needs`
|
||||||
|
# list of the `lint` job as this is what gates PRs.
|
||||||
lint:
|
lint:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
needs: [golangci-lint, yamllint, shellcheck]
|
needs: [golangci-lint, yamllint, shellcheck]
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue