Skip to content

feat: add lefthook pre-commit hooks for format/lint checks#89

Merged
mairas merged 1 commit intomainfrom
feat/issue-88-lefthook-precommit
Dec 17, 2025
Merged

feat: add lefthook pre-commit hooks for format/lint checks#89
mairas merged 1 commit intomainfrom
feat/issue-88-lefthook-precommit

Conversation

@mairas
Copy link
Contributor

@mairas mairas commented Dec 17, 2025

Summary

  • Add lefthook.yml with cargo fmt and clippy checks
  • Add hooks-install command to run script
  • Update README with developer setup instructions

Test plan

  • Tested lefthook run pre-commit locally
  • Verified run script hooks-install command works
  • Confirmed hooks run on commit

Developer Setup

After merging, developers should:

brew install lefthook
./run hooks-install

Fixes #88

🤖 Generated with Claude Code

- Add lefthook.yml with cargo fmt and clippy checks
- Add hooks-install command to run script
- Update README with developer setup instructions

Fixes #88

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@mairas
Copy link
Contributor Author

mairas commented Dec 17, 2025

Review: ✅ LGTM

CI Status: Passing ✓

Changes reviewed:

  • lefthook.yml - Standard Rust pre-commit config with workspace-aware flags
  • run script additions - Clean integration with existing script
  • README.md - Clear developer setup docs

Highlights:

  • Correctly uses cargo fmt --all -- --check for workspace support
  • Minimal, focused additions to the existing run script (21 lines)
  • Consistent with the broader lefthook rollout across HaLOS repos

Verdict: Ready to merge.

@mairas mairas merged commit 2c6b779 into main Dec 17, 2025
1 check passed
@mairas mairas deleted the feat/issue-88-lefthook-precommit branch December 17, 2025 09:53
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.

feat: add lefthook pre-commit hooks for format/lint checks

1 participant