Skip to content
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

Work around pypy38: skipped error, in newer tox and uv, in CI #336

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

john-kurkowski
Copy link
Owner

@john-kurkowski john-kurkowski commented Aug 20, 2024

Instead of #335's rollback of tool versions, trying to roll forward.

Changes

  • Drop PyPy 3.8 from CI
    • It's causing issues with uv in CI that aren't worth hoop jumping. PyPy 3.8 hasn't received updates in years. CPython 3.8's EOL is within 2 months.

TODO

  • Wait for upstream CPython 3.8 to be EOL 2024-10-31?
  • Remove noisy commits

@john-kurkowski john-kurkowski changed the title Work around pypy38: skipped error, in CI Work around pypy38: skipped error, in newer tox and uv, in CI Aug 21, 2024
Prefer to roll forward with newer versions of these tools.

This reverts commit b7f5aae.
Skipping missing interpreters became an error in newer versions of
tox-uv in particular, in 1 test runner, Windows PyPy 3.8. Other test
runners were unaffected. I'm not sure if the 1 affected test runner's
interpreter is actually missing. I suspect it's a bad check on the newer
tool versions' part. Either way, interpreters should _not_ be missing
and skipped in CI. _Not_ skipping missing interpreters causes CI to
succeed.
Trying a different approach.

This reverts commit 75a6bfc.
It's causing issues with `uv` in CI that aren't worth hoop jumping. PyPy 3.8 hasn't received updates in years. CPython 3.8's EOL is within 2 months.
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.

1 participant