Skip to content

Commit

Permalink
fix: code review findings
Browse files Browse the repository at this point in the history
Signed-off-by: Zsolt Rappi <[email protected]>
  • Loading branch information
rappizs committed Aug 31, 2023
1 parent 349a099 commit e096917
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ on:
jobs:
check-integrity:
name: Check integrity
runs-on: ubuntu-latest
runs-on: ubuntu-22.04

steps:
- name: Set up Go
Expand All @@ -22,16 +22,16 @@ jobs:

- name: Check Go modules dependency file integrity
run: |
for module_file in $(find . -type f -name go.mod); do
module=$(dirname $module_file)
cd "$module"
find . -type f -name go.mod | while read module_file; do
module=$(dirname "$module_file")
pushd "$module" > /dev/null
go mod tidy
if [ "$(git status --porcelain)" != "" ]; then
printf >&2 '\n`go mod tidy` in module `%s` results in a dirty state, Go mod files are not in sync with the source code files, differences:\n\n%s\n\n' "$module" "$(git diff)"
git reset --hard
exit 1
fi
cd - > /dev/null
popd > /dev/null
done
- name: Check generated file integrity
Expand All @@ -45,7 +45,7 @@ jobs:
build:
name: Build
runs-on: ubuntu-latest
runs-on: ubuntu-22.04

steps:
- name: Set up Go
Expand Down

0 comments on commit e096917

Please sign in to comment.