Skip to content

Conversation

@jontsai
Copy link
Contributor

@jontsai jontsai commented Oct 30, 2025

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:

error: the configured PyPy interpreter version (3.10) is lower than
PyO3's minimum supported version (3.11)

This is caused by the nh3 package (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 jobs

Removed Jobs

  • pypy310-pydantic28-cover
  • pypy310-pydantic210-cover
  • pypy310-pydantic28-nocov
  • pypy310-pydantic210-nocov

Impact

  • ✅ Fixes all failing CI builds
  • ✅ PyPy 3.9 jobs continue to work and remain in the matrix
  • ✅ All CPython versions (3.10, 3.11, 3.12) unaffected

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.

  • Behavior:
    • Remove PyPy 3.10 from tox.ini envlist and basepython definitions.
    • Remove PyPy 3.10 jobs from .github/workflows/github-actions.yml.
  • Removed Jobs:
    • pypy310-pydantic28-cover, pypy310-pydantic210-cover, pypy310-pydantic28-nocov, pypy310-pydantic210-nocov.
  • Impact:
    • Fixes failing CI builds due to PyO3 requiring PyPy 3.11+.
    • PyPy 3.9 and CPython versions (3.10, 3.11, 3.12) remain unaffected.

This description was created by Ellipsis for 5e92d2f. You can customize this summary. It will automatically update as commits are pushed.

Copy link

@ellipsis-dev ellipsis-dev bot left a 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 21 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 2 draft 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:
    Removed pypy310 from 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% <= threshold 50% None
2. tox.ini:25
  • Draft comment:
    Removed the pypy310 basepython 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% <= threshold 50% None

Workflow ID: wflow_1EZ1u5i8e301a3Sh

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@jontsai jontsai requested a review from suryaiyer95 October 30, 2025 22:21
@jontsai jontsai force-pushed the fix/remove-pypy310-from-tests branch from 130bc8c to 627cbee Compare November 6, 2025 23:59
Copy link

@ellipsis-dev ellipsis-dev bot left a 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 35 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 draft 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% <= threshold 50% None

Workflow ID: wflow_uiJI2edrtuyNqLO7

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@jontsai
Copy link
Contributor Author

jontsai commented Nov 7, 2025

✅ Fix Completed

Updated this PR to also remove PyPy 3.10 from the GitHub Actions workflow (.github/workflows/github-actions.yml).

Root Cause

All 4 PyPy 3.10 jobs were failing with:

error: the configured PyPy interpreter version (3.10) is lower than PyO3's minimum supported version (3.11)

This is caused by the nh3 package dependency requiring PyO3, which now requires PyPy 3.11+.

Changes

  • ✅ Removed from tox.ini (already done)
  • Now also removed from .github/workflows/github-actions.yml (new commit)

Removed Jobs

  • pypy310-pydantic28-cover
  • pypy310-pydantic210-cover
  • pypy310-pydantic28-nocov
  • pypy310-pydantic210-nocov

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]>
@jontsai jontsai force-pushed the fix/remove-pypy310-from-tests branch from 28b007d to 5e92d2f Compare November 7, 2025 00:18
Copy link

@ellipsis-dev ellipsis-dev bot left a 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 56 lines of code in 2 files
  • Skipped 0 files when reviewing.
  • Skipped posting 2 draft 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 using python: '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% <= threshold 50% 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 Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants