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

Drop support for Python 3.7 (not until January 2024) #10113

Closed
AlexWaygood opened this issue Apr 29, 2023 · 14 comments
Closed

Drop support for Python 3.7 (not until January 2024) #10113

AlexWaygood opened this issue Apr 29, 2023 · 14 comments
Assignees
Labels
stubs: removal Pending removal of third-party distributions

Comments

@AlexWaygood
Copy link
Member

Python 3.7 will reach end-of-life on 2023-06-27. For 3.6, we waited 6 months after EOL before dropping support. Do we want to do something similar this time, or are we happy with a shorter grace period?

3.7 is still a very popular Python version. On the other hand, large parts of the Python ecosystem have already dropped support for Python 3.7 in their latest versions (flake8, isort, numpy, pandas, Django).

@hauntsaninja
Copy link
Collaborator

6 months seems fine to me.

@JelleZijlstra
Copy link
Member

Agree, I don't think we need to hurry to drop old versions. The maintenance load from old versions in typeshed isn't that great, and lots of people still use 3.7.

@AlexWaygood
Copy link
Member Author

AlexWaygood commented Apr 29, 2023

I don't have a strong opinion either way, I just wanted to have the discussion. 6 months is fine for me!

Edit: Well, one reason I am looking forward to dropping support for 3.7 is so that we can use PEP 570 syntax in typeshed. But that doesn't justify breaking other projects, and can definitely wait.

@Avasam
Copy link
Collaborator

Avasam commented Apr 30, 2023

6 months seems reasonable, it's also in line with typeshed's already established time frame of supporting obsolete stubs. And just like obsolete stubs, older versions will still be available on PyPI.
I'm really looking forward to using PEP 570 and waiting for it to properly type pywin32's C-modules.

@AlexWaygood
Copy link
Member Author

I'm really looking forward to using PEP 570 and waiting for it to properly type pywin32's C-modules.

Hmm, can't you use the __ prefix for arguments? I expect we'll do some kind of codemod once we drop support for Python 3.7, that automatically converts the old-style syntax for positional-only arguments to PEP-570 syntax.

@Avasam
Copy link
Collaborator

Avasam commented Apr 30, 2023

Hmm, can't you use the __ prefix for arguments? I expect we'll do some kind of codemod once we drop support for Python 3.7, that automatically converts the old-style syntax for positional-only arguments to PEP-570 syntax.

Short answer is: I could, but I'd rather wait.

I didn't think of a codemod, so that removes the concern of having to do additional work to migrate.
But I still think it'll be easier to review if only a / is added in hundreds of methods (only a select few c-extension methods support named params), rather than all param names changing. It'll be easier and faster to do too.

@AlexWaygood AlexWaygood changed the title Python 3.7 EOL Drop support for Python 3.7 (not until January 2024) Jul 8, 2023
@AlexWaygood
Copy link
Member Author

AlexWaygood commented Jul 18, 2023

We're going to face an issue when mypy 1.5 is released, due to the fact that mypy v1.5 will not support being run on Python 3.7. This will break our stubtest-stdlib job on Python 3.7, which has to be run using Python 3.7 (the --python-version 3.7 flag will not suffice, as it needs to know what exists on Python 3.7 at runtime). It will also break our mypy_test.py job on Python 3.7 -- this also needs to be run using Python 3.7 (rather than just with the --python-version 3.7 flag), or, when typechecking tensorflow, the script will install the latest version of numpy -- a version of numpy that's incompatible with Python 3.7, and will cause mypy to emit errors in CI when run with the --python-version 3.7 flag.

We have four options that I can see:

  1. Don't bump our mypy pin until January 2024. This doesn't seem like a great idea.
  2. Use a different mypy pin when running mypy on Python 3.7 than when running mypy on Python 3.8+. Probably feasible, but pretty icky.
  3. Stop running stubtest on our py37 stubs; stop running mypy on our tensorflow stubs with Python 3.7
  4. Drop support for Python 3.7 earlier than January 2024 (in practice I guess this isn't too different from (3), since (3) involves ceasing to test with Python 3.7 in many respects).

@Avasam
Copy link
Collaborator

Avasam commented Jul 18, 2023

About your second suggestion:
Since requirements support specifying a mypy version, you could use that.

Even the grep command that gets the mypy version in Github Actions may not need much updating since passing in both lines/restrictions should be fine (one will always be skipped due to python version requirement)

@srittau
Copy link
Collaborator

srittau commented Jul 18, 2023

I think option 3 sounds reasonable to me. Our 3.7 support should be fairly stable by now, so we mainly lose regression tests for 3.7 running with Python. I'm okay with that.

@srittau
Copy link
Collaborator

srittau commented Jul 18, 2023

The main difference to option 4 is that we don't actively clean up our code by removing 3.7 branches etc.

@AlexWaygood
Copy link
Member Author

The main difference to option 4 is that we don't actively clean up our code by removing 3.7 branches etc.

Yes, we would "drop support" for Python 3.7 in that we wouldn't actively test with it in CI anymore, so we wouldn't be able to guarantee that our Python 3.7 stubs were bug-free. But we wouldn't remove the 3.7 branches or switch to using PEP-570 syntax etc. for another few months. That sounds okay to me as well.

@srittau
Copy link
Collaborator

srittau commented Aug 21, 2023

I wonder whether we should update our documents to claim that we only support Python 3.8+, but leave the branches in and don't use PEP 570 syntax until January. We can't really guarantee Python 3.7 support anymore and this can lead to confusion due to tests not being run etc. But inofficially we'd still support it.

@AlexWaygood
Copy link
Member Author

Note that we are still running py37 tests with pyright. But yeah, I agree that the current situation is somewhat confusing.

@srittau
Copy link
Collaborator

srittau commented Nov 10, 2023

How to remove support. (Please amend as necessary.)

Initial PRs

Subsequent PRs

@srittau srittau added stubs: removal Pending removal of third-party distributions and removed project: policy Organization of the typeshed project labels Jan 4, 2024
srittau added a commit to srittau/typeshed that referenced this issue Jan 4, 2024
@srittau srittau self-assigned this Jan 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stubs: removal Pending removal of third-party distributions
Projects
None yet
Development

No branches or pull requests

5 participants