Skip to content

Add prek-based pre-commit lint hook#595

Merged
maxachis merged 2 commits intodevfrom
fix-587-add-prek-lint-hook
Mar 9, 2026
Merged

Add prek-based pre-commit lint hook#595
maxachis merged 2 commits intodevfrom
fix-587-add-prek-lint-hook

Conversation

@maxachis
Copy link
Collaborator

Summary

  • add a prek pre-commit configuration that runs flake8 with existing plugin set
  • move flake8 ignore settings into shared .flake8 config and point reviewdog workflow at flake8 dot
  • add prek and flake8 tooling to the dev dependency group and update contributor/testing docs

Validation

  • uv lock
  • uv sync --group dev
  • uv run prek install
  • uv run prek run --all-files (fails due to pre-existing repository lint violations; hook wiring/config is functional)

Copy link
Contributor

@josh-chamberlain josh-chamberlain left a comment

Choose a reason for hiding this comment

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

we like pre-commit linting

Keep reviewdog approach since .flake8 config centralizes ignore settings,
replacing the inline --ignore flags from dev.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@maxachis maxachis merged commit c834e5d into dev Mar 9, 2026
3 of 4 checks passed
@maxachis maxachis deleted the fix-587-add-prek-lint-hook branch March 9, 2026 20:42
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