-
Notifications
You must be signed in to change notification settings - Fork 9
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
ES-1621: active forward merge automation from 5.1 -> 5.2 #1335
ES-1621: active forward merge automation from 5.1 -> 5.2 #1335
Conversation
Kudos, SonarCloud Quality Gate passed! |
Scanning for breaking API changes introduced by this PR Scan Succeeded |
Jenkins build for PR 1335 build 1 Build Successful: |
Jenkins build for PR 1335 build 2 Build Successful: |
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
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
This PR will activate forward merge automation between 5.1 and 5.2, reducing the need for manual merges. Only one PR is raised per target branch. at a time, any outstanding changes will be swept up in the next merge once PR A is merged.
PR Description generated by merge bot
The description of the PR will be automatically filled out with all pull requests carried forward in the merge.
PR Reviewers
The original authors of the pull requests will also be tagged in the forward merge so they can sanity-check their work.
Manual merges:
On occasion, you may not wish for a change to be brought forward if this is the case, on the original PR you can add the label
manual-merge
. The automation checks for the presence of this label and will take no action if it is found, your change will need to be manually merged forward, if required.Slack integration:
c5-forward-merge-bot-notifications
will auto-update by a bot user once a PR is ready to be merged