Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Drop support for running with Python 3.8#17492

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
hauntsaninja merged 3 commits intopython:masterfromcdce8p:drop-3.8
Dec 25, 2024

Conversation

@cdce8p
Copy link
Collaborator

Similar to last year (#15566), start by dropping support for running mypy with Python 3.8.
Users will still be able to type check 3.8 code with--python-version 3.8 until typeshed drops the support for it.

It's a bit early as the EOL for Python 3.8 is in ~3 months. However, since the branch for1.11.0 has been cut already, we'd only drop the support with1.12.0 which isn't due for another 1-2 months. Additionally dropping3.8 now will make it easier to support3.13 with its C-API changes and also give us enough time to cleanup the remaining 3.8 code blocks and documentation references.

@github-actions

This comment has been minimized.

1 similar comment
@github-actions

This comment has been minimized.

@cdce8p
Copy link
CollaboratorAuthor

cdce8p commentedJul 5, 2024
edited
Loading

Not sure why the Windows (>=3.9) uses the absolute file path when Linux and MacOS use the relative one. Maybe someone with a Windows system can help debug this?

@hamdanal
Copy link
Collaborator

Not sure why the Windows (>=3.9) uses the absolute file path when Linux and MacOS use the relative one. Maybe someone with a Windows system can help debug this?

Not sure why the difference either. The closest change in Python 3.9 related to this I could find is the second bullet point herehttps://docs.python.org/3/whatsnew/3.9.html#other-language-changes but it doesn't explain the difference in behavior between operating systems.

It is probably safe to just shorten the file paths in the output traceback before comparing it with the expected output. I tried this patch with the test case "mypyc/test/test_run.py::TestRun::run-loops.test::testForIterable" and it seems to fix the issue:

diff --git a/mypyc/test/test_run.py b/mypyc/test/test_run.pyindex 37de192a9..112074047 100644--- a/mypyc/test/test_run.py+++ b/mypyc/test/test_run.py@@ -315,6 +315,7 @@ class TestRun(MypycDataSuite):             # TODO: testDecorators1 hangs on 3.12, remove this once fixed             proc.wait(timeout=30)         output = proc.communicate()[0].decode("utf8")+        output = output.replace(f'  File "{os.getcwd()}{os.sep}', '  File "')         outlines = output.splitlines()         if testcase.config.getoption("--mypyc-showc"):

It is kind of a hack but I don't know how to fix it otherwise.

Off-topic: why not use a more recent python version like 3.12 or 3.11 for Windows tests so that we don't have to update it every year?

@cdce8p
Copy link
CollaboratorAuthor

Not sure why the Windows (>=3.9) uses the absolute file path when Linux and MacOS use the relative one. Maybe someone with a Windows system can help debug this?

Not sure why the difference either. The closest change in Python 3.9 related to this I could find is the second bullet point herehttps://docs.python.org/3/whatsnew/3.9.html#other-language-changes but it doesn't explain the difference in behavior between operating systems.

Thanks! Yeah, that seems to be the culprit. Pushed our patch as it's probably the best way forward here, especially since the path deviates depending on the OS used to run the tests.

Off-topic: why not use a more recent python version like 3.12 or 3.11 for Windows tests so that we don't have to update it every year?

I don't know the exact reason but I believe the intention is to always test the "oldest" Python release with Windows. Anyway, the issue mentioned earlier persists even in newer versions.That was one of the first things I checked.

AlexWaygood reacted with thumbs up emoji

@github-actions

This comment has been minimized.

Copy link
Collaborator

@hauntsaninjahauntsaninja left a comment
edited
Loading

Choose a reason for hiding this comment

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

Thanks, looks good. I think nice to drop support a little closer to 3.8 end of life, let's at least wait until the 1.11 release is out.

@JukkaL
Copy link
Collaborator

I think mypy 1.12 at least should probably still support 3.8. That way we might have a release that supports both 3.13 and 3.8. Runtime support for 3.8 is needed for projects that use mypyc and target 3.8, since it's not possible to run mypyc on a more recent Python version and target an older version.

@cdce8p
Copy link
CollaboratorAuthor

I think mypy 1.12 at least should probably still support 3.8. That way we might have a release that supports both 3.13 and 3.8.

One option of course. Though I'm not sure that's really necessary. We didn't do that the last time.3.7 was dropped with mypy1.5.0 whereas only1.6.0 added support for3.12. I suspect many projects are looking into dropping support for3.8 in the next few months anyway if they haven't already started doing so.

Avasam reacted with thumbs up emoji

@JukkaL
Copy link
Collaborator

Though I'm not sure that's really necessary.

Yeah it doesn't seem necessary, but if it doesn't take any significant effort, I think it's still worth doing. 3.8 still appears to be more popular than 3.9 or 3.12, based on PyPI stats (https://pypistats.org/packages/mypy).

cdce8p, Avasam, and edgarrmondragon reacted with thumbs up emoji

@hauntsaninja
Copy link
Collaborator

As of this week, Python 3.8 accounts for 10.5% of all mypy downloads and 4.8% of all downloads of mypy 1.11.*

cdce8p reacted with thumbs up emoji

@cdce8p
Copy link
CollaboratorAuthor

As of this week, Python 3.8 accounts for 10.5% of all mypy downloads and 4.8% of all downloads of mypy 1.11.*

Would you recommend waiting longer to drop 3.8? I'd say it's probably still fine to do after the 1.12 release. Users interested in new type checking features are more likely to be using newer Python versions.

Additionally, it will still be possible to lint 3.8 code (for the time being) and devs can always continue using 1.12 if they are stuck on 3.8.

@hauntsaninja
Copy link
Collaborator

hauntsaninja commentedOct 10, 2024
edited
Loading

Sorry, I should have added an opinion, I think it's fine to drop. 4.8% isn't a lot and it will drop further.
pypistats doesn't let you see Python version breakdown for a specific mypy version, I was curious how much less than the 10.5% it would be for 1.11.*

cdce8p reacted with thumbs up emoji

@JukkaL
Copy link
Collaborator

Let's make the decision when we are closer to 1.13 release? If there have been significant regressions or issues with new functionality in 1.12 (that we haven't fixed in a point release), we can extend 3.8 support to 1.13 so that there are more options for projects that want to support both 3.8 and 3.13. Or we can decide to have one more release supporting 3.8 if it's only minimal extra effort at that point.

JelleZijlstra reacted with thumbs up emoji

@mr-c
Copy link
Contributor

When this PR is closer to being merged, I suggest runningpyupgrade --py39-plus $(git ls-files | grep \.py$ | grep -v setup.py) , or as a follow-up PR, and storing the commit hash in.git-blame-ignore-revs

cdce8p reacted with thumbs up emoji

@cdce8p
Copy link
CollaboratorAuthor

When this PR is closer to being merged, I suggest runningpyupgrade --py39-plus $(git ls-files | grep \.py$ | grep -v setup.py) , or as a follow-up PR, and storing the commit hash in.git-blame-ignore-revs

👍🏻 Yeah, I have planned to do that. It's intentionally omitted here to keep the changes to a minimum.

mr-c reacted with heart emoji

@github-actions

This comment has been minimized.

@JukkaL
Copy link
Collaborator

We are aiming at having another release branch created in late October/early November. If we can stick to this schedule, I think we can manage another feature release with 3.8 support, and we'll drop support afterwards.

@cdce8p
Copy link
CollaboratorAuthor

We are aiming at having another release branch created in late October/early November. If we can stick to this schedule, I think we can manage another feature release with 3.8 support, and we'll drop support afterwards.

Sounds good 👍🏻 Just wanted to fix the merge conflicts here.

@github-actions

This comment has been minimized.

@JukkaL
Copy link
Collaborator

We can merge this after the release branch for 1.14 has been created.

cdce8p reacted with thumbs up emojimr-c and jack-mcivor reacted with hooray emoji

@github-actions
Copy link
Contributor

According tomypy_primer, this change doesn't affect type check results on a corpus of open source code. ✅

@hauntsaninja
Copy link
Collaborator

Eyeballing it looks like 3.8 usage has halved since October. Thanks for keeping this PR fresh!

@hauntsaninjahauntsaninja merged commit2d6b521 intopython:masterDec 25, 2024
19 checks passed
@cdce8pcdce8p deleted the drop-3.8 branchDecember 25, 2024 10:44
hauntsaninja pushed a commit that referenced this pull requestMay 28, 2025
Drop last remaining support for Python 3.8. Support for running with 3.8was removed in#17492 already. This PR removes the option to use 3.8with `--python-version` since the type stubs only support 3.9+, see#18930.
ilevkivskyi pushed a commit that referenced this pull requestNov 28, 2025
Similar to last year (#17492), start by dropping support for runningmypy with Python 3.9.Users will still be able to type check 3.9 code with `--python-version3.9` until typeshed drops the support for it.Refs#20154
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@hauntsaninjahauntsaninjahauntsaninja approved these changes

Assignees

No one assigned

Labels

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@cdce8p@hamdanal@JukkaL@hauntsaninja@mr-c

[8]ページ先頭

©2009-2025 Movatter.jp