Skip to content
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

misc(git3po): merge when green for unstable state too #12078

Merged
merged 1 commit into from
Feb 10, 2021

Conversation

connorjclark
Copy link
Collaborator

Unstable means all required checks passed, but some un required checks failed. For us, that means code cov (not required) can prevent our land-when-green label from working.

Only thing documenting unstable I could find: octokit/octokit.net#1763

@connorjclark connorjclark requested a review from a team as a code owner February 10, 2021 21:56
@connorjclark connorjclark requested review from adamraine and removed request for a team February 10, 2021 21:56
@google-cla google-cla bot added the cla: yes label Feb 10, 2021
@paulirish paulirish requested review from paulirish and removed request for adamraine February 10, 2021 22:13
@paulirish paulirish merged commit 3cf2521 into master Feb 10, 2021
@paulirish paulirish deleted the git3po-green-unstable branch February 10, 2021 22:14
@brendankenny
Copy link
Member

I feel like I've definitely seen it merge when green but non-required checks were red (like when travis was on until recently), but maybe something changed?

@paulirish
Copy link
Member

I feel like I've definitely seen it merge when green but non-required checks were red (like when travis was on until recently), but maybe something changed?

that's my guess. gh officially says this field is undocumented and indicates they can change behavior whenever. so perhaps they did.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants