-
Notifications
You must be signed in to change notification settings - Fork 1
Remove PyPy 3.10 from test matrix to fix failing CI checks #83
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
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.
Important
Looks good to me! 👍
Reviewed everything up to 130bc8c in 40 seconds. Click for details.
- Reviewed
21lines of code in1files - Skipped
0files when reviewing. - Skipped posting
2draft comments. View those below. - Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. tox.ini:17
- Draft comment:
Removedpypy310from the envlist as intended. Confirm that this removal fully covers cis testing needs for PyPy versions. Future changes in dependency requirements might require updating documentation. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%None
2. tox.ini:25
- Draft comment:
Removed thepypy310basepython configuration to align with dependency constraints. Ensure that no other parts of the CI configuration inadvertently reference this obsolete interpreter. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%None
Workflow ID: wflow_1EZ1u5i8e301a3Sh
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
130bc8c to
627cbee
Compare
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.
Important
Looks good to me! 👍
Reviewed 28b007d in 57 seconds. Click for details.
- Reviewed
35lines of code in1files - Skipped
0files when reviewing. - Skipped posting
1draft comments. View those below. - Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. .github/workflows/github-actions.yml:91
- Draft comment:
Removal of the PyPy 3.10 CI jobs is appropriate given the dependency issues with PyO3. This change aligns the CI matrix with the updated tox.ini. Ensure documentation is updated to reflect the removal. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%None
Workflow ID: wflow_uiJI2edrtuyNqLO7
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
✅ Fix CompletedUpdated this PR to also remove PyPy 3.10 from the GitHub Actions workflow ( Root CauseAll 4 PyPy 3.10 jobs were failing with: This is caused by the Changes
Removed Jobs
PyPy 3.9 jobs still work fine and remain in the test matrix. This should fix the failing CI builds. ✨ |
PyPy 3.10 jobs are failing because PyO3 (required by nh3 package) now requires PyPy 3.11 as the minimum version. Error from CI: error: the configured PyPy interpreter version (3.10) is lower than PyO3's minimum supported version (3.11) Changes: - Remove pypy310 from tox.ini envlist - Remove pypy310 basepython definition - Remove 4 pypy310 jobs from GitHub Actions workflow: - pypy310-pydantic28-cover - pypy310-pydantic210-cover - pypy310-pydantic28-nocov - pypy310-pydantic210-nocov PyPy 3.9 jobs continue to work and remain in the test matrix. Fixes failing CI builds on main branch. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
28b007d to
5e92d2f
Compare
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.
Important
Looks good to me! 👍
Reviewed 5e92d2f in 1 minute and 5 seconds. Click for details.
- Reviewed
56lines of code in2files - Skipped
0files when reviewing. - Skipped posting
2draft comments. View those below. - Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. .github/workflows/github-actions.yml:22
- Draft comment:
Removal of the PyPy 3.10 jobs is correctly handled here. The jobs with names starting with 'pypy310-' and usingpython: 'pypy-3.10'have been removed, which aligns with the dependency requirements. Ensure that any documentation referencing these jobs is updated accordingly. - Reason this comment was not posted:
Comment was not on a location in the diff, so it can't be submitted as a review comment.
2. tox.ini:17
- Draft comment:
The tox envlist has been updated to remove the PyPy 3.10 environments and the corresponding basepython mapping is removed. This change is consistent with the new PyO3 minimum version requirement. - Reason this comment was not posted:
Comment did not seem useful. Confidence is useful =0%<= threshold50%This comment is purely informative, explaining a change that was made without suggesting any action or asking for clarification. It doesn't provide any actionable feedback or raise any concerns about the code.
Workflow ID: wflow_xTYaD0j36DkrhRis
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
Summary
Remove PyPy 3.10 from the test matrix to fix failing CI checks.
Problem
All 4 PyPy 3.10 jobs are failing on every build with:
This is caused by the
nh3package (a dependency) requiring PyO3, which now requires PyPy 3.11+ as the minimum version.Solution
Remove PyPy 3.10 from both:
tox.ini- envlist and basepython definitions.github/workflows/github-actions.yml- all 4 pypy310 jobsRemoved Jobs
pypy310-pydantic28-coverpypy310-pydantic210-coverpypy310-pydantic28-nocovpypy310-pydantic210-nocovImpact
Testing
CI will run on this PR to verify the fix works.
Important
Remove PyPy 3.10 from test matrix to fix CI failures due to PyO3's PyPy 3.11+ requirement.
tox.inienvlist and basepython definitions..github/workflows/github-actions.yml.pypy310-pydantic28-cover,pypy310-pydantic210-cover,pypy310-pydantic28-nocov,pypy310-pydantic210-nocov.This description was created by
for 5e92d2f. You can customize this summary. It will automatically update as commits are pushed.