fix: exempt standalone E2E/data-loading scripts and forced bump#20
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review infoConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro 📒 Files selected for processing (2)
WalkthroughUpdates pre-commit hooks: replaces Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.pre-commit-config.yaml:
- Around line 27-31: The pre-commit hook for ruff (id: ruff-check) currently
sets language: system which bypasses the pinned rev (v0.15.4) and may cause
version drift; either remove the language: system line so pre-commit uses the
pinned rev v0.15.4 consistently, or if the intent is to use the system-installed
ruff, replace it with a one-line comment immediately above the ruff-check entry
stating that this is intentional and why (e.g., "Use system ruff to match dev
tooling managed by uv"); update only the ruff-check block referenced by id:
ruff-check to apply one of these two fixes.
- pyproject.toml: add per-file C901 ignore for test-e2e.py and load-sample-data.py (standalone scripts where high CC is expected) - .pre-commit-config.yaml: remove language/types/require_serial overrides from ruff-check (let pre-commit manage its own env); fix reuse-lint entry to use system reuse binary directly instead of uv run --no-sync which targeted .venv where reuse was absent
929dce1 to
34b384b
Compare
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
|
@coderabbitai review |
✅ Actions performedReview triggered.
|



Forced bump due to missed PR title earlier not triggering release.
Summary by CodeRabbit