Skip to content

prek/pre-commit: add rust formatting and checks#194

Merged
consideRatio merged 3 commits intosensmetry:mainfrom
consideRatio:pr/pre-commit-rust-formatting
Mar 5, 2026
Merged

prek/pre-commit: add rust formatting and checks#194
consideRatio merged 3 commits intosensmetry:mainfrom
consideRatio:pr/pre-commit-rust-formatting

Conversation

@consideRatio
Copy link
Copy Markdown
Collaborator

@consideRatio consideRatio commented Mar 4, 2026

This would have caught a mistake of mine in #192 where I failed to re-format.

Signed-off-by: Erik Sundell <erik.sundell+2025@sensmetry.com>
Copy link
Copy Markdown
Collaborator

@victor-linroth-sensmetry victor-linroth-sensmetry left a comment

Choose a reason for hiding this comment

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

If I understand correctly these commands will run where git commit is being run(?) so we should probably make them cover all the workspace.

Comment thread .pre-commit-config.yaml Outdated
Comment thread .pre-commit-config.yaml Outdated
Comment thread .pre-commit-config.yaml Outdated
Comment thread .pre-commit-config.yaml
Comment thread .pre-commit-config.yaml Outdated
Comment thread .pre-commit-config.yaml Outdated
Co-authored-by: Victor Linroth <victor.linroth@sensmetry.com>
Signed-off-by: Erik Sundell <erik.i.sundell@gmail.com>
Signed-off-by: Erik Sundell <erik.sundell+2025@sensmetry.com>
@consideRatio consideRatio force-pushed the pr/pre-commit-rust-formatting branch from f11eb1b to 968509d Compare March 4, 2026 14:35
Copy link
Copy Markdown
Collaborator

@victor-linroth-sensmetry victor-linroth-sensmetry left a comment

Choose a reason for hiding this comment

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

Looks good.

@consideRatio consideRatio enabled auto-merge March 5, 2026 08:49
@consideRatio consideRatio merged commit 2da8741 into sensmetry:main Mar 5, 2026
15 checks passed
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