Skip to content

ci: add pre-commit hooks with prek for code quality#767

Merged
dguido merged 1 commit intomasterfrom
dev/pre-commit-hooks
Jan 21, 2026
Merged

ci: add pre-commit hooks with prek for code quality#767
dguido merged 1 commit intomasterfrom
dev/pre-commit-hooks

Conversation

@dguido
Copy link
Copy Markdown
Member

@dguido dguido commented Jan 21, 2026

Summary

  • Add pre-commit configuration using prek (Rust-based, fast git hooks)
  • Configure hooks: goimports, go fmt, golangci-lint, dprint, actionlint
  • Add dprint.json for markdown/YAML/JSON formatting
  • Update flake.nix with required dev tools

This prevents import drift after the recent standardization in #764 (cc9a852).

Test plan

  • Run prek install to install hooks
  • Run prek run --all-files within nix develop shell to verify all hooks pass
  • Verify nix build still works

🤖 Generated with Claude Code

Add pre-commit configuration using prek (Rust-based, fast git hooks):
- goimports: format imports and verify no changes
- go fmt: format Go code and verify no changes
- golangci-lint: comprehensive lint checks
- dprint: format markdown, YAML, and JSON files
- actionlint: lint GitHub Actions workflows

Also adds dprint.json config and updates flake.nix with required
dev tools (golangci-lint, dprint, actionlint).

This prevents import drift after the recent standardization in cc9a852.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@dguido dguido merged commit d50e95b into master Jan 21, 2026
14 checks passed
@dguido dguido deleted the dev/pre-commit-hooks branch January 21, 2026 00:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant