[#1266] .forgejo: Make 'fumpt' job fail on changed files
All checks were successful
DCO action / DCO (pull_request) Successful in 1m4s
Tests and linters / Run gofumpt (pull_request) Successful in 59s
Vulncheck / Vulncheck (pull_request) Successful in 2m24s
Tests and linters / Staticcheck (pull_request) Successful in 2m27s
Pre-commit hooks / Pre-commit (pull_request) Successful in 2m43s
Build / Build Components (pull_request) Successful in 2m50s
Tests and linters / gopls check (pull_request) Successful in 3m2s
Tests and linters / Lint (pull_request) Successful in 3m10s
Tests and linters / Tests (pull_request) Successful in 4m36s
Tests and linters / Tests with -race (pull_request) Successful in 6m24s

`gofumpt` always returns an exit code of 0, even when it finds
misformatted files. To make `fumpt` action behave as expected
we need to check if `gofumpt` changed any files.

Signed-off-by: Ekaterina Lebedeva <ekaterina.lebedeva@yadro.com>
This commit is contained in:
Ekaterina Lebedeva 2024-09-13 15:44:23 +03:00
parent 89d0435b1d
commit 2be1aa781d

View file

@ -106,4 +106,6 @@ jobs:
run: make fumpt-install run: make fumpt-install
- name: Run gofumpt - name: Run gofumpt
run: make fumpt run: |
make fumpt
git diff --exit-code --quiet