-
Notifications
You must be signed in to change notification settings - Fork 518
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
[mergify] add backport policy and default label #6186
Conversation
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
Trends 🧪 |
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.
Nice, thanks @v1v
We can think about enforcing this rule to do not allow merges but for the time being, what do you think?
I think this is a good step. If the nudge isn't enough, then we can come back to blocking merges.
There is a missing bit regarding skipping the backports validation |
Motivation/summary
Notify PR contributors about the backport policy.
This policy will notify what are the different backport options and tag the PR with the
backport-skip
.We can think about enforcing this rule to do not allow merges but for the time being, what do you think?
Test
See elastic/apm-pipeline-library#1281 (comment) and elastic/apm-pipeline-library#1281 (comment)