-
Notifications
You must be signed in to change notification settings - Fork 481
ci: add tests for python3.14 #2138
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: main
Are you sure you want to change the base?
Conversation
Check if https://github.com/ag2ai/ag2/actions/runs/18352926389 could pass |
It doesn't relate to 3.14 promote - all tests are running with python3.10 |
PR Review: Add Tests for Python 3.14Thank you for working on Python 3.14 support! Here's my comprehensive review: Critical Issue - BlockingMissing requires-python constraint update The PR adds Python 3.14 to CI workflows and classifiers, but does not update the requires-python constraint in pyproject.toml:
Impact: The current constraint <3.14 explicitly excludes Python 3.14, meaning:
Action Required: Update line 27 in pyproject.toml to allow Python 3.14. Code Quality & Best PracticesPositive Aspects
Suggestions
Potential Issues
Performance Considerations
Security Concerns
Test CoverageCoverage is appropriate - the PR adds Python 3.14 to:
Exclusions properly maintained:
RecommendationsBefore merging:
After initial CI run:
Overall AssessmentThis is a well-structured PR with systematic updates across all CI workflows. However, it cannot be merged without updating the requires-python constraint, as this would create a broken configuration where CI tests Python 3.14 but the package explicitly excludes it. Recommendation: Request changes to fix the requires-python constraint before merging. |
* [skip ci] docs: fix outdated docs * Change < 15 to <=14 --------- Co-authored-by: joggrbot[bot] <107281636+joggrbot[bot]@users.noreply.github.com> Co-authored-by: Mark Sze <66362098+marklysze@users.noreply.github.com>
|
Code Review for PR #2138: Add Python 3.14 SupportSummaryThis PR adds Python 3.14 to the CI test matrix and updates documentation. The changes are straightforward and systematic, adding "3.14" to all relevant Python version matrices across the CI workflows. Positive Aspects
Critical Issues1. Python 3.14 Availability
2. Dependency Compatibility Issues
3. Test Status
RecommendationStatus: Approve with Changes Required The PR is well-structured but needs updates to dependency constraints before it can work correctly:
Review generated by Claude Code |
Codecov Report✅ All modified and coverable lines are covered by tests. 🚀 New features to boost your workflow:
|
No description provided.