-
Notifications
You must be signed in to change notification settings - Fork 277
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
[Retrospective] Release Version 2.11.1 #4259
Comments
We detected an issue and are working to avoid it to happen againSo, there was some kind of mess during the release process 😅. We realized that a bunch of non-bugfix patches had been merged in the #4223 was opened to discuss this and improve this point in the future. This is great. The community could discuss the issue and how to deal with itTo be honest, I was quite speechless at the very beginning when there was a proposal to "intentionally" ignore the SemVer promise and ship features in a patch release among other resolution plans in #4161. This was terrible. I am relieved the community decided to revert the wrongly merged PRs in order to get back on track 😄. Thanks to all the people who worked hard for this to happen 😍. This also is great! This took a lot of timeIf I try to do a quick timeline of events, I get:
So it took ~1 month to release a patch version when a critical issue was found. That does not look great. I understand that this is a large project with a lot of moving parts and a bunch of other fixes where merged. Also the above issue made the process way slower. But I understood branches like the |
@Divyaasm can we close this issue? |
Closing this issue as we have released 2.12.0 version. |
Related release issue?
#4161
How to use this issue?
Please add comments to this issue, they can be small or large in scope. Honest feedback is important to improve our processes, suggestions are also welcomed but not required.
What will happen to this issue post release?
There will be a discussion(s) about how the release went and how the next release can be improved. Then this ticket will be updated with the notes of that discussion along side action items.
The text was updated successfully, but these errors were encountered: