Skip to content

Commit

Permalink
Make sure to check for git dirty after gen/lint/tests
Browse files Browse the repository at this point in the history
  • Loading branch information
Frostman committed Dec 2, 2024
1 parent 223079b commit 23dd4f3
Showing 1 changed file with 8 additions and 9 deletions.
17 changes: 8 additions & 9 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -80,16 +80,9 @@ jobs:
# path: bin
# key: bin-${{ hashFiles('hack/tools.mk') }}

- name: GHA Lint
- name: Lint and generate code
run: |
just --timestamp lint-gha
# Don't remove it! We shouldn't allow CI to pass if there are any changes not committed after running just
- name: Just all autogeneration and fail on dirty
run: |
just --timestamp gen
git status --short
git diff --quiet
just --timestamp lint-gha gen _lint
- name: Some Go lint
run: |
Expand All @@ -105,6 +98,12 @@ jobs:
run: |
just --timestamp test
# Don't remove it! We shouldn't allow CI to pass if there are any changes not committed after running just
- name: Fail on dirty
run: |
git status --short
git diff --quiet
- name: Setup tmate session for debug
if: ${{ failure() && github.event_name == 'workflow_dispatch' && inputs.debug_enabled }}
uses: mxschmitt/action-tmate@v3
Expand Down

0 comments on commit 23dd4f3

Please sign in to comment.