-
Notifications
You must be signed in to change notification settings - Fork 2
CI: repo hygiene checks #601
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
a2f75b2 to
1f194a9
Compare
|
Code coverage report:
|
| with: | ||
| cache: true | ||
| go-version: ${{ steps.tool_versions.outputs.go_version }} | ||
| go-version-file: 'go.mod' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
misc cleanup in yaml files
| arch=$(uname -m) | ||
|
|
||
| install_dir=/usr/local | ||
| $install_dir/bin/protoc --version | grep $VERSION |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
misc shellcheck fixes
| docker run --rm \ | ||
| -v "${PWD}:/local" \ | ||
| openapitools/openapi-generator-cli generate \ | ||
| openapitools/openapi-generator-cli:v7.19.0 generate \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pin version so that the VERSION file doesn't change unexpectedly.
| run: just install-go-tools | ||
|
|
||
| - name: Run repo hygiene checks | ||
| run: tools/bin/check_repo_clean.sh tidy mock generate shellcheck |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New CI command. Runs the provided just targets and ensures they don't modify the repo or return an error.
Automation to ensure go mods, mocks and openapi generated files are all up to date.