Skip to content

Add pre-commit CI hooks - #13

Merged
lucarlig merged 1 commit into
mainfrom
fix/add-pre-commit-ci-hooks
May 14, 2026
Merged

Add pre-commit CI hooks#13
lucarlig merged 1 commit into
mainfrom
fix/add-pre-commit-ci-hooks

Conversation

@lucarlig

Copy link
Copy Markdown
Contributor

Summary

  • Add a local pre-commit configuration that mirrors CI: fmt, clippy, nextest, build, and bench compile.
  • Disable filename passing so each hook runs at workspace scope like GitHub Actions.

Validation

  • cargo fmt --all --check
  • cargo clippy --locked --workspace --all-targets -- -D warnings
  • cargo nextest run --locked --workspace
  • cargo build --locked --workspace
  • cargo bench --locked --workspace --no-run
  • uvx --from pre-commit pre-commit validate-config
  • git commit hook reran all five configured hooks successfully

@lucarlig
lucarlig requested a review from dawid-nowak May 13, 2026 15:10
@lucarlig
lucarlig marked this pull request as ready for review May 13, 2026 15:10

@dawid-nowak dawid-nowak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Signed-off-by: lucarlig <luca.carlig@ibm.com>
@lucarlig
lucarlig force-pushed the fix/add-pre-commit-ci-hooks branch from 1db77bb to f510aa0 Compare May 13, 2026 16:32
@lucarlig
lucarlig merged commit 2c4c1be into main May 14, 2026
6 checks passed
@lucarlig
lucarlig deleted the fix/add-pre-commit-ci-hooks branch May 14, 2026 08:04
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.

2 participants