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

Support Gerrit Verify Status REST endpoints #60

Closed
wants to merge 1 commit into from

Conversation

zaro0508
Copy link

@zaro0508 zaro0508 commented Sep 7, 2016

The gerrit-verify-status plugin provides additional Gerrit REST
endpoints. This adds support for those endpoints[1].

This resolves issue #59

[1] https://gerrit.googlesource.com/plugins/verify-status/+/master/src/main/resources/Documentation/rest-api-changes.md

The gerrit-verify-status plugin provides additional Gerrit REST
endpoints. This adds support for those endpoints[1].

This resolves issue uwolfer#59

[1] https://gerrit.googlesource.com/plugins/verify-status/+/master/src/main/resources/Documentation/rest-api-changes.md
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.9%) to 92.111% when pulling 9c2ef65 on zaro0508:resolve-issue-#59 into 57567cb on uwolfer:master.

@uwolfer
Copy link
Owner

uwolfer commented Oct 29, 2016

Thanks for your PR and sorry for my late feedback.

I have not merged it yet since I'm not sure how we should handle code which depends on Gerrit plugins. It might be confusing for users of this plugin when they see this API, but cannot use it with every server.

I'll put this change on hold for the moment.

@uwolfer uwolfer force-pushed the master branch 7 times, most recently from 105caaf to 91cd9fc Compare November 3, 2018 09:32
@zaro0508 zaro0508 closed this May 25, 2022
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.

3 participants