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

fix "Unknown content type null" in logs and since for 1.12.0 api #66

Merged
merged 6 commits into from
Jul 29, 2015

Conversation

lanwen
Copy link
Member

@lanwen lanwen commented Jul 29, 2015

  • some small changes (see commits) to help implementing future prs

Review on Reviewable

@@ -118,6 +118,8 @@ protected GHRepository adapt(GitHub item) {
*
* This is useful if the caller only relies on the read access to the repository and doesn't need to
* walk possible candidates.
*
* Result can be null
*/
public GHRepository resolveOne() {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

annotate with CheckForNull

@@ -118,7 +119,10 @@ protected GHRepository adapt(GitHub item) {
*
* This is useful if the caller only relies on the read access to the repository and doesn't need to
* walk possible candidates.
*
* Result can be null
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove this line, checkfornull is enough or if you want java doc then change to @return desc

KostyaSha added a commit that referenced this pull request Jul 29, 2015
fix "Unknown content type null" in logs and since for 1.12.0 api
@KostyaSha KostyaSha merged commit a8dc555 into jenkinsci:master Jul 29, 2015
@lanwen lanwen deleted the misc branch July 29, 2015 16:45
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.

2 participants