Match all merge triggers instead of any

[philBrown]

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

merge:
  trigger:
    labels: ["dependencies"]
    branches: ["main"]
    branch_patterns: ["dependabot/.*"]

however I noticed that Bulldozer is merging other PRs with the following message

my-repo#129 is triggered because triggering is enabled and pull request target is a triggered branch: "main"

This PR had no labels and was not within a dependabot/* branch name.

Is there any way to only trigger when all conditions match?

[philBrown]

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

[asvoboda]

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?