Skip to content

chore: add a pre-commit config adapted from jebel-quant/rhiza - #53

Merged
tschm merged 2 commits into
mainfrom
chore_pre_commit_config_20260925
Sep 25, 2026
Merged

tschm merged 2 commits into
mainfrom
chore_pre_commit_config_20260925

Conversation

@tschm

@tschm tschm commented Sep 25, 2026

Copy link
Copy Markdown
Member

Adds .pre-commit-config.yaml, adapted from jebel-quant/rhiza for a repo that isn't rhiza-managed and keeps its Python project in collector/. The header comment records each change:

  • Dropped: rhiza-hooks (they validate .rhiza/), the template-bundles and renovate schema checks (those files don't exist here), and interrogate (without a [tool.interrogate] table it would use its own defaults, a standard nobody here chose).
  • Rescoped: bandit to collector/jq_collector/ (rhiza reads a .bandit file this repo doesn't have), uv-lock to --project collector, and validate-pyproject to collector/pyproject.toml.
  • Pinned: ruff to v0.16.9, the version in collector/uv.lock, so the hook and CI's Lint step can't disagree.

It also includes the one change ruff-format made when first run: two blank lines removed in scripts/check-dashboard.py. CI's Format step only covers jq_collector tests, which is why this drift was never caught.

The hooks don't all pass yet (uvx prek run --all-files): 11 pass, 2 fail.

Nothing runs these hooks in CI yet, so merging this changes no check. Wiring them in is a follow-up once these failures are cleared.

🤖 Generated with Claude Code

@tschm
tschm merged commit e974c0c into main Sep 25, 2026
5 checks passed
@tschm
tschm deleted the chore_pre_commit_config_20260925 branch September 25, 2026 13:10
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.

1 participant