-
Notifications
You must be signed in to change notification settings - Fork 43
Update release policy page #1447
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
Conversation
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.
Commented below. Everything else does not contradict with my understanding of the processes.
@kyukhin Please, decide whether this outstanding situation should be handled this way (via publishing a new release out of usual timelines). Since our repositories have per-push packages, it is not necessary, maybe. However we should notify users about necessity to update and properly describe reasons. |
Maybe just notify users about the patch. I see no reason for tagging new release in the case. |
@veod32 Please, adjust the cited sentence accordingly. |
@Totktonada All review comments have been resolved. Please review once again. |
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 for the changes!
I looked over the resulting document and highlighted everything that contradits with my understanding or may be formulated better.
@Totktonada I've resolved all the comments. Also I restructured the content and re-wrote some of the paragraphs as we discussed. Some of the paragraphs are deleted as the contained the duplication of info. |
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 gave several minor comments, but they are tiny and there is no need to re-review it with me.
Maybe @kyukhin want to take a glance at the final variant.
Co-authored-by: Elena Shebunyaeva <elena.shebunyaeva@gmail.com>
Co-authored-by: Elena Shebunyaeva <elena.shebunyaeva@gmail.com>
Co-authored-by: Elena Shebunyaeva <elena.shebunyaeva@gmail.com>
No description provided.