chore: update dependencies#79
Conversation
|
Warning Rate limit exceeded
⌛ 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. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
WalkthroughUpdated project Python requirement from Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes 🚥 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. 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
🧹 Nitpick comments (1)
pyproject.toml (1)
23-24: Move non-runtime dependencies from[project.dependencies]to[dependency-groups]dev.
markdown-captionsis not registered in the markdown extension setup (onlycodehilite,fenced_code,toc,meta,tables, andBootstrapExtensionare used inapi.py).poethepoetis exclusively used for developer task configuration in[tool.poe.tasks]and is not imported in any source code. Both should be moved to the dev dependency group.Proposed dependency placement update
[project] dependencies = [ @@ - "markdown-captions>=2.1.2", - "poethepoet", ] [dependency-groups] dev = [ + "poethepoet", "ipython", "pip-audit", "pytest", "ruff (>=0.12.3)", ]🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@pyproject.toml` around lines 23 - 24, Remove "markdown-captions" and "poethepoet" from the top-level [project.dependencies] and add them into the development dependency group (e.g., [project.dependency-groups.dev.dependencies]) so they are treated as dev-only dependencies; keep their version specifiers unchanged and ensure [tool.poe.tasks] continues to work with poethepoet listed under the dev group and that markdown-captions is not expected as a runtime dependency in api.py.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@pyproject.toml`:
- Line 9: The declared requires-python = ">=3.10" in pyproject.toml doesn't
match the CI test matrix; either update the CI workflow to include Python 3.10
and 3.11 (expand the test matrix to 3.10, 3.11, 3.12) or change requires-python
to ">=3.12" to reflect the single tested interpreter; modify the
pyproject.toml's requires-python or the CI job matrix (whichever you choose) so
the declared Python support and the CI matrix are consistent.
---
Nitpick comments:
In `@pyproject.toml`:
- Around line 23-24: Remove "markdown-captions" and "poethepoet" from the
top-level [project.dependencies] and add them into the development dependency
group (e.g., [project.dependency-groups.dev.dependencies]) so they are treated
as dev-only dependencies; keep their version specifiers unchanged and ensure
[tool.poe.tasks] continues to work with poethepoet listed under the dev group
and that markdown-captions is not expected as a runtime dependency in api.py.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 9ca8c54f-c78d-4fcf-a464-c283ce2abb7e
⛔ Files ignored due to path filters (1)
uv.lockis excluded by!**/*.lock
📒 Files selected for processing (1)
pyproject.toml
Co-Authored-by: Pascal Repond <pascal.repond@rero.ch>
No description provided.