-
Notifications
You must be signed in to change notification settings - Fork 554
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
1.12.0 release notes don't mention breaking changes to fix vulnerability #2253
Comments
Here's some guidance:
|
A good note for this might be:
EDIT: I'm cribbing @asraa's wording |
Thank you Will update the release notes with this |
Great, thanks for the quick fix @cpanato 😄 Now, the long-run question: does our existing release notes tooling handle security advisory fixes? Is there a process fix here? I can imagine the same thing happening for any fix to an advisory. |
we will improve in the release notes for sure, and i think in this case was a miscommunication :( |
will close this |
The v1.12.0 release contains a fix for GHSA-8gw7-4j42-w388 which is technically a breaking change (something that used to work no longer does).
While the advisory is called out at the top, it may not be clear to someone reading the release notes that this might cause issues without clicking through. Further, there's no guidance for how to fix it if their workflow was affected.
CC @cpanato @puerco any ideas on process fixes here?
The text was updated successfully, but these errors were encountered: