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

Release 0.630 planning #5537

Closed
9 of 12 tasks
ilevkivskyi opened this issue Aug 29, 2018 · 25 comments
Closed
9 of 12 tasks

Release 0.630 planning #5537

ilevkivskyi opened this issue Aug 29, 2018 · 25 comments
Assignees

Comments

@ilevkivskyi
Copy link
Member

ilevkivskyi commented Aug 29, 2018

I would like to release mypy 0.630 on Friday September 7 September 14. The release branch is going to be was cut on Tuesday September 4 Friday September 5. Please mention here any issues and PRs that you would like to include in the release.

Here is the list of release blockers:

@ilevkivskyi
Copy link
Member Author

I think these two should be fixed before the release: #5534 and #2128

@ethanhs
Copy link
Collaborator

ethanhs commented Aug 29, 2018

I would like to get #5517 in, as it makes finding typeshed a lot simpler and more robust.

@Michael0x2a
Copy link
Collaborator

I was just about to mention #2128, actually -- I'm currently in the process of writing some code to resolve that issue and hope to submit a PR either later today or sometime tomorrow.

@srittau
Copy link
Contributor

srittau commented Aug 29, 2018

Is there any chance to have #4913 fixed in the release? Guido's comment seems to indicate it is an easy fix, although I can't judge this.

@gvanrossum
Copy link
Member

Maybe #4567? It's a plugin hook for MRO calculation that some user would like to see. I think it's unobjectionable, though it currently has a non-trivial conflict (you edited the same code in your most recent fix or MRO-related issues).

@ilevkivskyi
Copy link
Member Author

I think it also makes sense to fix #5514

@JelleZijlstra
Copy link
Member

In general, it would be good to sync typeshed right before creating the branch, so that people's typeshed fixes make it in. Maybe that could be part of the standard release procedure.

@ilevkivskyi
Copy link
Member Author

Maybe that could be part of the standard release procedure.

This is in our internal team playbook. I will make a sync on Monday probably.

@gvanrossum
Copy link
Member

A regression: #5557.

@Michael0x2a
Copy link
Collaborator

This is cutting it close to the deadline (sorry!), but it'd be nice if #5571 could also be merged in. (It partially addresses some of the feedback regarding error messages in #5557).

@ilevkivskyi
Copy link
Member Author

#5573 should be fixed before the release.

@gvanrossum
Copy link
Member

I suspect #5576 may be worth fixing?

At this point we should probably consider postponing the release by a few business days.

I'd also like to see a consolidated list of all issues and PRs that are blocking the release.

@ilevkivskyi
Copy link
Member Author

I'd also like to see a consolidated list of all issues and PRs that are blocking the release.

I updated the description with a list.

I suspect #5576 may be worth fixing?

Why is this important? This release is already late, I would limit ourselves to only critical things.

At this point we should probably consider postponing the release by a few business days.

Yes, I think we can do it on Monday-Tuesday. However, if there are no additional things I would cut release branch tomorrow.

@gvanrossum
Copy link
Member

OK, I see #5576 is not that important -- it's not a regression AFAICT.

@gvanrossum
Copy link
Member

Then it looks like there are no blockers left. Can you cut the branch tonight?

@ilevkivskyi
Copy link
Member Author

Then it looks like there are no blockers left. Can you cut the branch tonight?

Hm, indeed. OK, I will do this now.

@ilevkivskyi
Copy link
Member Author

OK, release branch is cut.

@ilevkivskyi
Copy link
Member Author

After some thinking we decided that we should first fix #5514 and postpone the release by few days again -- new target date is Friday, September 14.

(I updated the issue description.)

@srittau
Copy link
Contributor

srittau commented Sep 13, 2018

Please have a look at python/typeshed#2456 as well. (Although I am not sure whether the changes causing this are in the release branch.)

@ilevkivskyi
Copy link
Member Author

@srittau The PR that caused this didn't get into the release branch, but we anyway decided to revert it.

@Michael0x2a
Copy link
Collaborator

I'm not sure if we've started the release process yet or not, but if there's still time, could we perhaps sneak in #5595? It's almost entirely a docs change -- the only actual change in the code is a small tweak to the text we print out when running mypy --help.

If this is too late/too much trouble, that's totally fine -- this is a pretty low-priority thing. I'm bringing this up mostly because I think it would be a little cleaner if we could include all of the command line/config file stuff in one release instead of across two. (So this PR would be released along with #5333 and #5334.)

@ilevkivskyi
Copy link
Member Author

We are kind of deep into the release process, in the sense that everything is ready, but the actual release was already postponed twice for some reasons. If everything will be OK, I will upload packages to PyPI on Monday morning. There is no big need for doc updates, since the website is anyway built from master. I mean you can already see your changes on https://mypy.readthedocs.io.

@Michael0x2a
Copy link
Collaborator

Only if you visit https://mypy.readthedocs.io/en/latest/ -- if you visit https://mypy.readthedocs.io/en/stable/ (which is what mypy-lang.org links to), you won't see the changes.

But anyways, no worries -- I figured I might as well try asking.

@ilevkivskyi
Copy link
Member Author

ilevkivskyi commented Sep 17, 2018

The release is out!

@gvanrossum
Copy link
Member

gvanrossum commented Sep 17, 2018 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants