Prek #149
Prek #149
Conversation
dosubot
bot
added
the
size:S
CoMPaTech
added
enhancement
|
Warning Rate limit exceeded@CoMPaTech has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 9 minutes and 40 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (4)
WalkthroughReplaces pre-commit with prek in the verify GitHub Actions workflow, updates environment variable and steps, bumps project version to 0.6.2, adds a changelog entry, and adds an explicit empty additional_dependencies for the markdownlint hook. Changes
Sequence Diagram(s)(omitted — changes are CI/workflow/tooling updates and not suitable for a multi-component runtime sequence diagram) Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 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 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #149 +/- ##
=======================================
Coverage 87.07% 87.07%
=======================================
Files 8 8
Lines 743 743
=======================================
Hits 647 647
Misses 96 96 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
dosubot
bot
added
the
size:S
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
.github/workflows/verify.yml (1)
6-6: UpdatePRE_COMMIT_HOMEtoPREK_HOMEfor consistency with prek.While prek respects
PRE_COMMIT_HOMEthrough compatibility fallbacks, prek usesPREK_HOME(default~/.cache/prek) as its native cache directory. Update line 6 toPREK_HOME: ~/.cache/prekto align with prek's conventions and make the tooling intent explicit.
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
.github/workflows/verify.ymlCHANGELOG.mdpyproject.toml
🧰 Additional context used
🪛 LanguageTool
CHANGELOG.md
[grammar] ~9-~9: Ensure spelling is correct
Context: ... ### Changed - Migrated pre-commit to prek ## [0.6.1] - 2026-01-03 ### Added - Provi...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Process test coverage
🔇 Additional comments (3)
CHANGELOG.md (1)
5-9: LGTM!The changelog entry is correctly formatted, follows the existing style, and accurately documents the migration from pre-commit to prek. The static analysis spelling warning for "prek" is a false positive as it's a tool name.
pyproject.toml (1)
7-7: LGTM!Version bump to 0.6.2 aligns with the changelog entry and is appropriate for a tooling change (patch version increment).
.github/workflows/verify.yml (1)
78-83: LGTM!The migration from pre-commit to prek is complete and consistent:
- Package installation updated correctly
- Hook installation command updated
- Run command preserves the same flags (
--show-diff-on-failure,--color=always,--all-files)- The
SKIP=no-commit-to-branchbypass is maintained
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
.github/workflows/verify.yml (1)
74-84: Removeprek install --install-hooksfrom CI or add explanatory comment if intentional.The
prek install --install-hookscommand pre-creates hook environments and installs git hooks, which is not needed for CI hook execution. Standard CI practice is to simply runprek run --all-fileswithout the install step. If this line serves a specific purpose (e.g., pre-warming environments), add a comment clarifying why. All flags (--show-diff-on-failure,--color,--all-files) andSKIP=no-commit-to-branchare fully supported by prek.
🧹 Nitpick comments (1)
.github/workflows/verify.yml (1)
3-7: Document the intentionalPREK_HOMEoverride for clarity.
PREK_HOMEis deliberately set to~/.cache/pre-commitrather than prek's documented default (~/.cache/prek). While this likely serves a purpose (cache reuse), add a comment explaining the rationale to prevent confusion for future maintainers. Shell expansion of~works as expected in GitHub Actions, so no change needed there.
dosubot
bot
added
size:M
|
Summary by CodeRabbit
Chores
Documentation
✏️ Tip: You can customize this high-level summary in your review settings.