-
Notifications
You must be signed in to change notification settings - Fork 29
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
Fix backport branch name is not correct #69
Fix backport branch name is not correct #69
Conversation
Signed-off-by: Hailong Cui <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #69 +/- ##
=========================================
Coverage 81.76% 81.76%
Complexity 86 86
=========================================
Files 8 8
Lines 521 521
Branches 70 70
=========================================
Hits 426 426
Misses 54 54
Partials 41 41 ☔ View full report in Codecov by Sentry. |
Backport is failing, do we need to handle this failure? |
line 39 will handle this. line12-21 will disable backport workflow run when creating a normal PR. |
Signed-off-by: Hailong Cui <[email protected]> (cherry picked from commit 05c8fe0) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* Fix backport branch name is not correct (#69) Signed-off-by: Hailong Cui <[email protected]> (cherry picked from commit 05c8fe0) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * include fix in #70 Signed-off-by: zhichao-aws <[email protected]> --------- Signed-off-by: Hailong Cui <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Signed-off-by: zhichao-aws <[email protected]> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: zhichao-aws <[email protected]>
…oject#71) * Fix backport branch name is not correct (opensearch-project#69) Signed-off-by: Hailong Cui <[email protected]> (cherry picked from commit 05c8fe0) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * include fix in opensearch-project#70 Signed-off-by: zhichao-aws <[email protected]> --------- Signed-off-by: Hailong Cui <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Signed-off-by: zhichao-aws <[email protected]> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: zhichao-aws <[email protected]> Signed-off-by: yuye-aws <[email protected]>
Description
The correct backport branch name should be something like
backport/backport-41-to-2.x
otherwise, backport branch will not get deleted when merged by
delete_backport_branch
workflowIssues Resolved
An example of wrong backportn name PR #68
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.