Repository navigation
chore: add go mod tidy to hk/lint checks - #1462
Conversation
There was a problem hiding this comment.
Pull request overview
Adds a go mod tidy verification step to the repositoryβs hk hook configuration so module metadata stays consistent during hk check --all / uds run lint runs.
Changes:
- Introduces a new
go-mod-tidyhook step that checksgo mod tidycleanliness viago mod tidy -diff. - Adds a corresponding auto-fix command (
go mod tidy) and scopes execution to Go/module files.
π‘ Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
bradclawsie
left a comment
There was a problem hiding this comment.
seems fine but what is our intent for updating pkgs? I usually do go mod tidy to its own PR, done only periodically
renovate already does this in it's prs (runs go mod tidy). however if you add a feature that adds a dep/bumps it we have no lint check as it currently stands to do this (i caught this with my import work). this should be a check and we should never have any non tidy go mod/sum files checked in |
Description
Adds
go mod tidycheck to hk/lint checks.Type of change
Checklist before merging