ReviewOS

also looking at this

stacks/buddy

fix(pr): apply the security label so security-only can fire

#1409
Merged glennmichael123 wants to merge fix/apply-the-security-label into main
4 files +87 -60
docs/features/auto-merge.mdmodified+1-1
Changes to docs/features/auto-merge.md
@@ -34,7 +34,7 @@ A PR qualifies when **any** listed condition accepts it.
3434| `security-only` | The PR carries the configured `security.label` (default `security`) |
3535| `all` | Any Buddy PR, including major upgrades |
3636
37`security-only` reads a label, and Buddy does not apply that label on its own — a PR gets it because you put it there, either through `pullRequest.labels` or a package rule. Configure `security-only` without arranging for the label and nothing will ever match it.
37`security-only` reads a label, and Buddy applies it to any pull request whose updates resolve a published advisory — matched against OSV.dev, not against package names that merely sound security-related. Rename it with `security.label`. You can also add that label by hand or through a package rule, and the condition treats it the same way.
3838
3939Unknown condition names are rejected by config validation rather than ignored, so a typo like `patch_only` fails loudly instead of widening what merges unreviewed.
4040