Skip to content

fix: remove duplicate cargo-audit hook, dogfood commit traceability#9

Merged
avrabe merged 1 commit intomainfrom
fix/pre-commit-hooks
Mar 13, 2026
Merged

fix: remove duplicate cargo-audit hook, dogfood commit traceability#9
avrabe merged 1 commit intomainfrom
fix/pre-commit-hooks

Conversation

@avrabe
Copy link
Contributor

@avrabe avrabe commented Mar 10, 2026

Summary

  • Remove duplicate cargo-audit entry in .pre-commit-config.yaml
  • First commit with artifact trailers — dogfooding commit traceability

Bootstrap

To use the commit-msg hook locally:

cargo install --path rivet-cli    # build rivet first
pre-commit install --hook-type commit-msg --hook-type pre-commit

Test plan

  • rivet validate passes
  • rivet commit-msg-check validates trailers
  • Pre-commit hooks installed and working

🤖 Generated with Claude Code

Remove duplicate cargo-audit entry in .pre-commit-config.yaml.
Verify pre-commit hooks are installed for commit-msg and pre-commit.

Implements: DD-011, DD-013
Implements: FEAT-029, FEAT-030, FEAT-031
Satisfies: REQ-017, REQ-018, REQ-019
Implements: DD-012, FEAT-032
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@codecov
Copy link

codecov bot commented Mar 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@avrabe avrabe merged commit bb50581 into main Mar 13, 2026
15 of 16 checks passed
@avrabe avrabe deleted the fix/pre-commit-hooks branch March 13, 2026 17:29
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