Skip to content

chore(quality): pre-commit hooks#38

Open
pentaxis93 wants to merge 2 commits intomainfrom
issue-12/pre-commit-hooks
Open

chore(quality): pre-commit hooks#38
pentaxis93 wants to merge 2 commits intomainfrom
issue-12/pre-commit-hooks

Conversation

@pentaxis93
Copy link
Copy Markdown
Owner

Summary

  • Add .pre-commit-config.yaml with ruff (check + format) and standard quality hooks (trailing-whitespace, end-of-file-fixer, check-yaml, check-added-large-files)
  • Update CONTRIBUTING.md with pre-commit setup instructions
  • Fix existing trailing whitespace and missing EOF newlines across the codebase (auto-fixed by hooks)

Changes

Commit 1 — chore(quality): add pre-commit hooks
New .pre-commit-config.yaml with pinned versions (pre-commit-hooks v6.0.0, ruff-pre-commit v0.15.5). CONTRIBUTING.md updated with install/run instructions.

Commit 2 — style: fix trailing whitespace and missing end-of-file newlines
14 files auto-fixed by running pre-commit run --all-files on the existing codebase. Pure whitespace — no functional changes.

Issue(s)

Closes #12

Test plan

  • pre-commit run --all-files passes clean
  • Ruff check and format hooks fire on Python files under server/
  • Standard hooks (trailing-whitespace, end-of-file-fixer, check-yaml, check-added-large-files) fire on all tracked files
  • Hook versions are pinned (not latest or floating)
  • CONTRIBUTING.md documents setup steps

pentaxis93 and others added 2 commits March 11, 2026 07:12
Add .pre-commit-config.yaml with ruff (check + format), trailing-whitespace,
end-of-file-fixer, check-yaml, and check-added-large-files hooks. Update
CONTRIBUTING.md with pre-commit setup instructions.

All hook versions pinned to specific releases:
- pre-commit-hooks v6.0.0
- ruff-pre-commit v0.15.5

Refs #12

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Auto-fixed by pre-commit hooks (trailing-whitespace, end-of-file-fixer)
running against the existing codebase.

Refs #12

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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.

chore(quality): pre-commit hooks

1 participant