-
Notifications
You must be signed in to change notification settings - Fork 97
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
Match all merge triggers instead of any #225
Comments
After having a look in https://github.com/palantir/bulldozer/blob/develop/bulldozer/signals.go#L52, I can see that this is simply not possible without significant refactoring 😢 Consider this a wishful feature request |
Hey @philBrown, Thanks for the comment. Usually we suggest pairing bulldozer with policy-bot. Policy-bot can be used to create complex approvals (for allow specific dependabot PRs) which can then be auto merged with Bulldozer. This is mentioned in the README, but perhaps we should make this a little more visible? |
While configuring Bulldozer, I assumed that a PR would need to match all triggers before being automatically merged.
For example, I'm using Bulldozer to auto-merge PRs from Dependabot with the following config
however I noticed that Bulldozer is merging other PRs with the following message
This PR had no labels and was not within a
dependabot/*
branch name.Is there any way to only trigger when all conditions match?
The text was updated successfully, but these errors were encountered: