-
Notifications
You must be signed in to change notification settings - Fork 203
Python 3.13 support #2078
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Python 3.13 support #2078
Conversation
|
👋 @vyagubov |
WalkthroughPython upper bound in Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 2
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Disabled knowledge base sources:
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
pyproject.toml(1 hunks)
🔇 Additional comments (1)
pyproject.toml (1)
16-16: Verify the networkx version constraint and test Python 3.13 compatibility for packages with relaxed upper bounds.Most dependencies support Python 3.13, but the networkx constraint needs clarification: if set to
<3, it would allow v2.x which is incompatible with Python 3.13. Additionally, verify that pydantic is constrained to v2.8+ (not lower v2 versions) and that actual testing confirms runtime compatibility across the entire dependency tree.
|
@arbiv @NoyaOffer |
|
Thanks @vyagubov, will do |
See: #1993
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.