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

#77: Approve PR that passed quality gate (Bitbucket Server) #114

Closed
wants to merge 1 commit into from

Conversation

artemy-osipov
Copy link
Contributor

A parameter has been added that allows to approve Pull Request that passed quality gate. If new changes are made and the quality gate no longer passes then approve is removed.

resolve #77 for Bitbucket Server

@marvin-w
Copy link
Contributor

I think this breaks all my changes xD

#105

dayum.

@artemy-osipov
Copy link
Contributor Author

@marvin-w Bitbucket Server and Bitbucket Cloud have incompatible API which requires a separate implementation of the PR decoration so I think the only conflict will be the name of the package.

@mc1arke
Copy link
Owner

mc1arke commented Jul 17, 2020

@artemy-osipov are you still interested in getting this merged? There have been quite a few changes in the Bitbucket implementation so this has conflicts which prevent me reviewing it properly and/or merging.

@artemy-osipov
Copy link
Contributor Author

Decorating PR for Bitbucket Server was migrated to Code Inside that has the functionality to restrict PR to merge. So it is no longer necessary to approve PR

@nfalco79
Copy link

It could be anyway a confgurable option. We had implemented in our pipeline with curl and wait to get the analisysId. If it was already included in the plugin since it has the bitbucket Bitbucket/GitHub/Lab authentication was 10000 better.

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

Successfully merging this pull request may close these issues.

Approve PR that passed quality gate
4 participants