You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We had recently at least 2 instances when not all Required Checks passed but were still merged. With automatic merge on (CI: Ready to merge label) we had:
In those examples some PR checks failed, one manually triggered a re-run because of flakiness. Then one cancelled the run and the label got picked up and PR merged.
PRs should never be allowed to be merged with required checks not being run successfully.
The text was updated successfully, but these errors were encountered:
We had recently at least 2 instances when not all Required Checks passed but were still merged. With automatic merge on (
CI: Ready to merge
label) we had:In those examples some PR checks failed, one manually triggered a re-run because of flakiness. Then one cancelled the run and the label got picked up and PR merged.
PRs should never be allowed to be merged with required checks not being run successfully.
The text was updated successfully, but these errors were encountered: