-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
add metric: "Require deployments to succeed before merging" #82
Comments
I see a |
It's possible this isn't in the API yet; i'll inquire. |
Oh yea nvm. "Require status checks to pass before merging" is a different metric. |
@ljharb do you know when we could have this metric included into the API rest? This is a great feature to insert in any workflow working with environments. |
@lmartinezviewnext the "on hold" label means it's not in github's api. as soon as Github adds it, we can add it here. |
API is not available for this feature and it's not possible to select an environment if it's not deployed on that environment. environment list is only visible once it's deployed on that env for the first time. |
@KamranBiglari thanks for confirming. we could still require an environment name here, and to satisfy it, you'd have to successfully deploy it before requiring it, just like a branch protection. |
This was also confusing for me as well. We are creating a step-by-step for setting up a brand new repository and would like to have this rule setup. The way it functions now requires us to create the repo, deploy it and then set the rule. Kind of backwards, no? |
This checkbox is under branch protections:
The text was updated successfully, but these errors were encountered: