- Notifications
You must be signed in to change notification settings - Fork673
add only_allow_merge_if_build_succeeds option to project objects#155
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
add only_allow_merge_if_build_succeeds option to project objects#155
Uh oh!
There was an error while loading.Please reload this page.
Conversation
The patch looks good, thank you. Please fix the pep8 failure and I'll merge. |
It didn't work =( I tried:
and
but none worked. Am I doing something wrong? |
I tested here and everything seems to work, using a boolean value. Are you sure that your gitlab version support this attribute? It's recent. |
rafaeleyng commentedSep 29, 2016 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
I can set this value using the GUI, but I'm not sure about the API support. Since I'm no Python developer, I may have done something wrong. Here is what I did:
Does this suffice to use the newest version containing the change introduced in this MR? |
Yes you should be able to use your change. Do you know the version of the gitlab server? |
From what I see in the git log you need gitlab 8.12 to get this feature in the API. |
We are using GitLab Community Edition 8.9.0. I'll try to update and give you some input again. |
No description provided.