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

Prevent typing_extensions 4.7.0 #2676

Merged
merged 5 commits into from
Jul 2, 2023

Conversation

A5rocks
Copy link
Contributor

@A5rocks A5rocks commented Jun 30, 2023

This is making our static tool tests fail.

@codecov
Copy link

codecov bot commented Jun 30, 2023

Codecov Report

Merging #2676 (a3fa648) into master (ae200e6) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2676   +/-   ##
=======================================
  Coverage   98.84%   98.84%           
=======================================
  Files         114      114           
  Lines       16514    16514           
  Branches     3003     3003           
=======================================
  Hits        16323    16323           
  Misses        134      134           
  Partials       57       57           

@A5rocks
Copy link
Contributor Author

A5rocks commented Jun 30, 2023

Oh duh, CI doesn't read the actual requirements file, only the locked one. And for the locked one, pip-tools doesn't realize that it should drop the cpython-only bit?

Anyways looks like typing_extensions might make a bugfix release. Otherwise I'll uh, idk, xfail the test? We'll see in a day.

Copy link
Member

@jakkdl jakkdl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, just mark it as skip and we can merge it and go ahead with merging other PRs. We don't need to care about whether this gets fixed upstream or not

@jakkdl
Copy link
Member

jakkdl commented Jul 1, 2023

And for the locked one, pip-tools doesn't realize that it should drop the cpython-only bit?

if you remove/comment out the first time typing_extensions is listed on line 26 it works. Playing around with it, it does not seem to be able to generate multiple different requirements in requirements.txt for the same package even if different lines in test-requirements.in are explicitly mutually exclusive.

@Fuyukai
Copy link
Member

Fuyukai commented Jul 1, 2023

Why are we even testing PyPy 3.7? As far as I can tell they're not even releasing 3.7 or 3.8 builds anymore.

@jakkdl
Copy link
Member

jakkdl commented Jul 1, 2023

Why are we even testing PyPy 3.7? As far as I can tell they're not even releasing 3.7 or 3.8 builds anymore.

we're dropping 3.7 support soon #2668 - so it's just a matter of it not having been dropped yet.

@Fuyukai
Copy link
Member

Fuyukai commented Jul 1, 2023

But PyPy support is different to CPython support. PyPy dropped 3.8 already, and it dropped 3.7 quite a while ago.

@jakkdl
Copy link
Member

jakkdl commented Jul 1, 2023

Was surprisingly tricky to find what versions are supported, but yeah looks like it's 3.9 and 3.10. I'll drop pypy-3.8 as well in #2668 and add pypy-3.10 as well, or do that in a different PR in case it's messy.

@A5rocks
Copy link
Contributor Author

A5rocks commented Jul 1, 2023

I'm just going to wait a little before merging this to see if typing extensions releases a bugfix to fix this (they have a pr open on their side and I think they will?)

@A5rocks
Copy link
Contributor Author

A5rocks commented Jul 1, 2023

Why are we even testing PyPy 3.7? As far as I can tell they're not even releasing 3.7 or 3.8 builds anymore.

Let's see what the PyPy maintainers recommend testing against! Here's where the question was asked: https://foss.heptapod.net/pypy/pypy/-/issues/3958#note_299941

@A5rocks
Copy link
Contributor Author

A5rocks commented Jul 2, 2023

Well, they merged in the bugfix 6 hours ago... but haven't made a release. Time window's up!

@A5rocks A5rocks merged commit c758fbc into python-trio:master Jul 2, 2023
@A5rocks A5rocks deleted the fix-failing-pypy-3.7 branch July 2, 2023 01:40
@A5rocks
Copy link
Contributor Author

A5rocks commented Jul 2, 2023

@jakkdl let's revert this in #2668

Additionally, could you update the README in that to say we're only going to support versions of PyPy PyPy itself still supports? (Namely 3.9 and 3.10 atm) See: https://foss.heptapod.net/pypy/pypy/-/issues/3958#note_299944

@jakkdl
Copy link
Member

jakkdl commented Jul 3, 2023

@jakkdl let's revert this in #2668

Additionally, could you update the README in that to say we're only going to support versions of PyPy PyPy itself still supports? (Namely 3.9 and 3.10 atm) See: https://foss.heptapod.net/pypy/pypy/-/issues/3958#note_299944

It does currently say:

Cool, but will it work on my system? Probably! As long as you have
some kind of Python 3.8-or-better (CPython or currently maintaned PyPy3 are
both fine), and are using Linux, macOS, Windows, or FreeBSD, then Trio
will work.

but I reworded it slightly

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.

3 participants