Skip to content

PR 3: Add pre-commit hooks for formatting, linting, and tests #111

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
natefikru wants to merge 4 commits into from
Closed

PR 3: Add pre-commit hooks for formatting, linting, and tests #111

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
62c48c4
Add pre-commit hook for formatting and clippy checks
natefikru Mar 26, 2026
210d2cf
Add tests to pre-commit hook and remove stderr suppression
natefikru Mar 30, 2026
90e93c4
Address review feedback: use make lint, remove tests and check target…
natefikru Mar 30, 2026
5f326f5
Update scripts/pre-commit
natefikru Mar 31, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,3 +43,9 @@ examples: build
cargo run -p turnkey_examples --bin sub_organization
cargo run -p turnkey_examples --bin wallet
cargo run -p turnkey_examples --bin proofs

.PHONY: install-hooks
install-hooks:
@cp scripts/pre-commit .git/hooks/pre-commit
@chmod +x .git/hooks/pre-commit
@echo "Pre-commit hook installed."
22 changes: 22 additions & 0 deletions scripts/pre-commit
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
#!/usr/bin/env bash
# Pre-commit hook for rust-sdk
# Runs formatting and clippy for staged Rust changes.
# Install with: make install-hooks

set -euo pipefail

# Check if any Rust files are staged
STAGED_RS=$(git diff --cached --name-only --diff-filter=ACM | grep '\.rs$' || true)
STAGED_TOML=$(git diff --cached --name-only --diff-filter=ACM | grep 'Cargo\.toml$' || true)

# No Rust files staged, skip checks
if [ -z "$STAGED_RS" ] && [ -z "$STAGED_TOML" ]; then
echo "No relevant changes. Skipping rust pre-commit checks."
exit 0
fi
Comment thread
natefikru marked this conversation as resolved.

echo "Running pre-commit checks..."

make lint

echo "Pre-commit checks passed."
Loading