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
GitHub status api #98
Conversation
It seems fine to me. Can you please rebase? |
Merged intermediate commits. |
Nice! How can it be tested? |
/** | ||
* post comment for github pull request | ||
*/ | ||
private void postComment(Pull pull, int lineNo, String comment, String commitSha, String filename) { |
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.
This is not called anywhere?
import java.io.StringWriter; | ||
import java.io.Writer; | ||
|
||
public class ContentRenderer { |
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.
maybe it could be package protected?
a5a25b2
to
cc38085
Compare
cc38085
to
6e098cf
Compare
HOST("connector.host", "Connector server host", "localhost"), | ||
PORT("connector.port", "Connector server port", "80"), | ||
PATH("connector.path", "Connector server path", ""), | ||
USE_HTTPS("connector.useHttps", "Connector use https?", "false"), | ||
USERNAME("connector.username", "Connector server username", "user"), | ||
PASSWORD("connector.password", "Connector server password", "password"), | ||
PROJECT_KEY("connector.projectKey", "Connector server projectKey", null), | ||
REPOSITORY_SLUG("connector.repositorySlug", "Connector server repositorySlug", null), | ||
OWNER("connector.owner", "Connector server owner", null), |
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 know it was some time ago and this PR is merged, but this change will break clients who have connector.repositorySlug
in their configuration files.
Moreover, not all elements have been changed: https://github.com/TouK/sputnik/search?utf8=%E2%9C%93&q=%22connector.repositorySlug%22+%22connector.projectKey%22&type=Code
GitHub status api
Github connector for Sputnik.
There is still an issue with jcabi-github. Waiting for jcabi/jcabi-github#1102 being merged.