Skip to content

feat: configure pre-commit hooks for cargo fmt and cargo clippy#16

Merged
Queenode merged 3 commits into
Kolo-Org:mainfrom
Jopsan-gm:issue-3-pre-commit-hooks
Jun 19, 2026
Merged

feat: configure pre-commit hooks for cargo fmt and cargo clippy#16
Queenode merged 3 commits into
Kolo-Org:mainfrom
Jopsan-gm:issue-3-pre-commit-hooks

Conversation

@Jopsan-gm

Copy link
Copy Markdown
Contributor

Title feat: configure pre-commit hooks for cargo fmt and cargo clippy

Description

This PR addresses issue #3 by setting up and configuring pre-commit hooks to automate code formatting and linting.

Closes #3

Changes

  • Added .pre-commit-config.yaml containing hooks for cargo fmt and cargo clippy.
  • Updated the root README.md with step-by-step instructions on how to install and configure pre-commit locally.
  • Formatted and resolved pre-existing warnings in the Rust files (contracts/src/lib.rs and contracts/src/test.rs) to ensure the pre-commit hooks run and pass cleanly.
  • Resolved a minor cargo warning by changing dev_dependencies to dev-dependencies in contracts/Cargo.toml.

Screenshots / Evidence

Below is the screenshot showing that the pre-commit hooks execute and pass successfully on commit:

image

@Jopsan-gm

Copy link
Copy Markdown
Contributor Author

Hi @Queenode, PR ready for review! ;)

@Queenode

Copy link
Copy Markdown
Contributor

Thank you for your contribution @Jopsan-gm .
Welldone

@Queenode
Queenode merged commit da2c664 into Kolo-Org:main Jun 19, 2026
1 check 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.

[Tooling] Configure pre-commit Hooks

2 participants