-
Notifications
You must be signed in to change notification settings - Fork 496
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
Release note targeting for multiple branches #17
Comments
Can the release note generator detect rolled back changes? On Fri, Jun 24, 2016 at 5:23 PM, David McMahon notifications@github.com
|
@erictune No. I had considered adding that, but then our canonical method for exposing release notes is through labeling and that single interface provides more flexibility in deciding what should or should not be captured as a release note (rollback spanned releases, rollback itself was relevant for capturing the fact that a feature didn't make it in a release, etc). Adding more moving parts here might confuse things. |
Issues go stale after 30d of inactivity. Prevent issues from auto-closing with an If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or |
/sig release |
Stale issues rot after 30d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or |
Rotten issues close after 30d of inactivity. Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
* Stop building kubefed * Adding kubefed.yaml back
There's no way to target different release note states for multiple branches due to the fact that the release note state is based on the one source PR, not the individual cherry-picks.
For example, #27861 seems like a good release note for 1.3 but we don't want it as a relnote for 1.2.5 (since it was rolled back).
The fix probably entails checking the cherry-pick for a release note state and letting it override the source PRs state and then updating the cherry-pick doc to allow for that in these special circumstances. I can't immediately think of any issues with that approach.
It will mean more processing time and github polling to check labels for both PRs for every merge pull.
cc @eparis @roberthbailey
The text was updated successfully, but these errors were encountered: