-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Release 1.0 planning #13685
Comments
I wasn't able to access. I requested access from |
Same for me (jelle.ziijlstra@gmail.com). I think the biggest risk here is that we commit to too many things before 1.0 and end up not releasing anything for a long time. That's bad for users because it means the improvements that do make it into master won't get to users. So I'm wary of committing to too many things that must be done before 1.0. |
I didn't see any objections, so here is a Zoom link I scheduled for Monday, Sep 26, 5PM Dublin time. (I also added access to the doc for everyone who requested, let me know if it still doesn't work for some people) |
That's 9am Pacific. |
The Zoom link asks for a passcode that I don't have. |
Hm, weird, can you try |
Are you clicking on the Zoom link above? It includes the password. |
That works, but you need to let us out of the waiting room ;) |
Btw, one thing we didn't discuss is documentation. |
I just merged One other change to get in for 1.0 we didn't mention: show error codes by default #13542 |
Btw we forgot to discuss one idea in the meeting: how do people feel about having separate versioning for plugin API? Like every time we have a breaking change, we bump the plugin API version, so that plugin authors can check it and give a better error message instead of crashing. |
Showing error codes by default sounds good to me. We probably still give the misc error code too often still, and we may want to switch some of them to non-misc error codes. At least the most common messages probably have reasonable error codes already, so the breakage from tweaking error codes in the future is probably acceptable. |
Ref #13685 Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
It might make sense to change the default branch name to |
It looks like 1.0 is coming closer and closer. I think we should aim to release 1.0 in first half of December. Overall it looks quite good in terms of 3.11 support and fixing crashes and top bugs. Few questions:
Thanks! |
Yes! Side question: during the last meeting we decided to have some regular calls. |
I think it should be done but I probably won't have time to work on it. |
I think there are no specific plans on these. |
I will try to work on this tomorrow/this weekend. If I don't get to it then, I probably will not have time to work on it for a while. |
@Michael0x2a It is almost December, so to be able to release mypy before holidays, I think you should start working on internal mypy pin move at Dropbox now (if you didn't already). This may discover some bugs/regressions that we will need to fix. Also please feel free to assign me to any issue caused by one of my PRs (but even if it is not my, I can still take a look). |
I wouldn't include dataclass_transform in 1.0, since the implementation is still incomplete and we haven't had much time to play around with it. The bug fixes seem good to include. Also fixing #14565 would be nice but I agree that we don't need to block the release on it. |
I'll try to have a look at #14150 soon to decide if we should include it in 1.0. I'm on the fence about including it at the last minute, since it's quite a big change, but on the other hand if we'd have it, the overall performance improvement in 1.0 would be more impressive. |
I just cherry picked a bunch of docs changes onto the release branch (and one testing change) |
We believe we have all of the necessary changes in the release-1.0 branch (speak now if you know that's not the case). We'll aim to release early next week. |
mypy 1.0.0 has been released (blog post)! If we need to do any point releases for bugfixes, I will announce it on this issue. |
Just tried it and ran it and got an error when I tried it on the python-gitlab/python-gitlab#2481 Logfile: https://github.com/python-gitlab/python-gitlab/actions/runs/4107691730/jobs/7087478909
|
@JohnVillalovos thanks for the report. Please report any problems you encounter in new issues. With that said, your issue isn't anything to do with mypy 1.0. |
Ping @ymyzk, please add to mypy-play.net :) |
Thanks for the heads-up, @intgr. mypy-play.net now supports mypy 1.0.0 Congratulations to everyone who made the 1.0.0 release possible. 🎉 |
Here's a list of reported issues thus far, in priority order, that would be worth fixing in a patch release:
|
I can look at the first and the third later this week (If no one else does it before). |
I looked at the first, should be an easy fix (on the lines of some other fix you made recently) Edit: PR here #14642 |
Reopening for the bugfix release. I think we should wait a few more days in case more bugs surface. I've sent out the PR #14646 to fix the used-before-def issue above. |
#14629 is a regression in the "user experience" of mypy, in that mypy now emits a false-positive error on code where it previously didn't. But strictly speaking this speaks to the absence of a feature in mypy 1.0, rather than a regression in mypy's functionality. The reason why mypy now emits an error on that code when it didn't previously is because mypy previously had no understanding of |
#14580 will also cause new false-negatives and false-positives because |
+1, we should not cherry pick it. Note that while #14629 has a new error, overall mypy 1.0 will complain less about the code in their example (e.g. mypy will let them instantiate their objects now) |
Another potential PR to cherry-pick (very safe since this is a test-only change): |
Looks like most of the important fixes are in and cherry-picked to release-1.0 branch. I'll make a 1.0.1 release tomorrow, unless anyone objects. |
mypy 1.0.1 has been released with the latest commit from release-1.0 branch! Going to close this issue and will reopen if we need to do another point release (although hopefully not :P). |
This commit will update to mypy 1. In the past, releases were managed by Dropbox (python/mypy#12210), and were done at infrequent intervals. They plan to release more regularly in the future. Mypy does not keep a changelog, but did provide a 1.0 release blog post: https://mypy-lang.blogspot.com/2023/02/mypy-10-released.html A changelog has been requested (python/mypy#2859, python/mypy#13279), but has been dismissed because "you can view the git history easily." The new release versioning scheme is in the format `major.minor.patch`, but is not SemVer-compliant, as explained in their blog post and wiki. https://github.com/python/mypy/wiki/Release-Process The update to mypy 1 was postponed until bug fixes were available in the 1.0.1 patch release (python/mypy#13685).
This commit will update to mypy 1. In the past, releases were managed by Dropbox (python/mypy#12210), and were done at infrequent intervals. They plan to release more regularly in the future. Mypy does not keep a changelog, but did provide a 1.0 release blog post: https://mypy-lang.blogspot.com/2023/02/mypy-10-released.html A changelog has been requested (python/mypy#2859, python/mypy#13279), but has been dismissed because "you can view the git history easily." The new release versioning scheme is in the format `major.minor.patch`, but is not SemVer-compliant, as explained in their blog post and wiki. https://github.com/python/mypy/wiki/Release-Process The update to mypy 1 was postponed until bug fixes were available in the 1.0.1 patch release (python/mypy#13685).
It is decided to release mypy 1.0 in ~two months, as mypy is mature enough for such a milestone. We need to plan this well in advance, so I am already opening this issue.
The result of some preliminary discussions is captured in this doc (please let me know if you have access issues). I would propose to have a Zoom call on Monday Sep 26 afternoon Dublin time (unless there are objections) to discuss the release plans in more details.
If you have any comments, questions, ideas, please leave them here (note we will still have 0.990 release between 0.980 and 1.0).
cc @JukkaL @jhance @JelleZijlstra @hauntsaninja @AlexWaygood @Michael0x2a @sobolevn @ethanhs @ilinum @svalentin (also please tag anyone I forgot here)
Tasks
More important things (in arbitrary order):
--enable-incomplete-features
Unpack
andTypeVarTuple
as possible @jhanceimplicit_optional
off by default (with a code transform tool to fix existing annotations) @hauntsaninjaEnum
)--namespace-packages
the default @hauntsaninjaLess important things:
--local-partial-types
enabled by default (if fallout is not bad)The text was updated successfully, but these errors were encountered: