-
Notifications
You must be signed in to change notification settings - Fork 362
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
chore: add backport github action #9835
Conversation
✅ Deploy Preview for determined-ui canceled.
|
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.
My comment aside, I think this is looking good. I'm not familiar with Github Actions though, so we ought to have a second set of eyes besides me.
jobs: | ||
backport_and_merge: | ||
runs-on: ubuntu-latest | ||
# Only react to merged PRs for security reasons. |
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.
Doesn't this comment belong a few lines lower?
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.
This section was copy/paste'd from https://github.com/tibdex/backport/blob/main/.github/workflows/backport.yml, but I can move the comment around
Ticket
CM-446
Description
See https://hpe-aiatscale.atlassian.net/wiki/x/CgBLZQ for more details.
Suppose a developer wants to backport a fix
XYZ
into release branchrelease-A
. On the pull request forXYZ
, add labelsbackport release-A
. Then, after the PR is merged, the github action will trigger :(1) creating a new backport PR from main into the release branch (in this case
release-A
)(2) automatically merging the backport PR
Test Plan
Manual testing done on https://github.com/carolinaecalderon/github-backport
Checklist
docs/release-notes/
See Release Note for details.