-
Notifications
You must be signed in to change notification settings - Fork 462
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
added new api call for forced push checks #1071 #1072
Conversation
with the new 6.0 release the api change may be okay, since everyone expects to do minor changes on major version change |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The change looks good to me, but the changes described in the comments are needed
changed the reported issues, would be great if you could check and merge into master for next major release |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made some additional changes.
I will perform some manual tests and merge
Here a change to expose the forced commit check
Gitlab documentation: https://docs.gitlab.com/ee/api/protected_branches.html
Fixes #1071