-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Cut 0.33.0-rc.0 #6841
Cut 0.33.0-rc.0 #6841
Conversation
fb08caa
to
b311715
Compare
b311715
to
664f5a5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
If this is what needs to be done. I guess there was a bot that writes all the changes into the changelog, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd say to make these changes directly to main
but they'll eventually merge back, so it should be fine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 to have this change in main, but doesn't block this pr
@@ -23,7 +23,7 @@ Release shepherd responsibilities: | |||
|
|||
| Release | Time of first RC | Shepherd (GitHub handle) | | |||
|---------|----------------------|-------------------------------| | |||
| v0.33.0 | (planned) 2023.10.03 | TBD | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this change is causing the docs check to fail
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm - the change to the VERSION doc, which is causing CI to fail
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Just need to fix the docs check.
I think it might be a bit more convenient to prep main
first for v0.33 via a PR like: #6617
Otherwise, you might get a bunch of merge conflicts
Signed-off-by: Michael Hoffmann <mhoffm@posteo.de>
664f5a5
to
98875c7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Let's get this rolling.
No description provided.