-
Notifications
You must be signed in to change notification settings - Fork 303
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
Add changelog for 6.1.0 #768
Add changelog for 6.1.0 #768
Conversation
0fc2a98
to
df01d45
Compare
@consideRatio, I think we might want to also fix this #758 as part of a new release. A workaround we've used for this was https://github.com/2i2c-org/infrastructure/pull/2865/files. |
Ah, I agree! |
df01d45
to
b234162
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.
I've resolved the merge conflicts, and want to appreciate you doing all the release prep work, @consideRatio! I'm happy to share some of the load if you so choose. But this is ready to go now from my perspective.
I've not pushed on this as i felt we absolutely need a note in the changelog to help users remediate the consequences of the bug as well - this is the place to make such note. I plan to test a script by Min tomorrow helping with that, possibly make a smaller tweak, and then link to it from the changelog. |
@consideRatio thanks for the update! |
e552ac2
to
5b8ae81
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.
LGTM
I propose that we cut a release now with bugfixes, a performance improvement, and the
unlisted_choice
functionality part ofprofile_lists
->profile_options
.If approved by monday next week, I'll cut a release then.