Merge pull request jenkinsci#132 from jenkinsci/lanwen-patch-1

lanwen · web-flow · commit 6fdefd9ac60d · 2016-07-21T20:41:13.000+03:00
add some override annotations in ghcommitnotifier
diff --git a/src/main/java/com/cloudbees/jenkins/GitHubCommitNotifier.java b/src/main/java/com/cloudbees/jenkins/GitHubCommitNotifier.java
@@ -108,6 +108,7 @@ public static Result getDefaultResultOnFailure() {
         return Result.fromString(trimToEmpty(resultOnFailure));
     }
 
+    @Override
     public BuildStepMonitor getRequiredMonitorService() {
         return BuildStepMonitor.NONE;
     }
@@ -153,10 +154,12 @@ public void perform(@NonNull Run<?, ?> build,
     @Extension
     public static class DescriptorImpl extends BuildStepDescriptor<Publisher> {
 
+        @Override
         public boolean isApplicable(Class<? extends AbstractProject> aClass) {
             return true;
         }
 
+        @Override
         public String getDisplayName() {
             return GitHubCommitNotifier_DisplayName();
         }

Original file line number	Diff line number	Diff line change
`@@ -108,6 +108,7 @@ public static Result getDefaultResultOnFailure() {`
`108`	`108`	`return Result.fromString(trimToEmpty(resultOnFailure));`
`109`	`109`	`}`
`110`	`110`
	`111`	`+ @Override`
`111`	`112`	`public BuildStepMonitor getRequiredMonitorService() {`
`112`	`113`	`return BuildStepMonitor.NONE;`
`113`	`114`	`}`
`@@ -153,10 +154,12 @@ public void perform(@NonNull Run<?, ?> build,`
`153`	`154`	`@Extension`
`154`	`155`	`public static class DescriptorImpl extends BuildStepDescriptor<Publisher> {`
`155`	`156`
	`157`	`+ @Override`
`156`	`158`	`public boolean isApplicable(Class<? extends AbstractProject> aClass) {`
`157`	`159`	`return true;`
`158`	`160`	`}`
`159`	`161`
	`162`	`+ @Override`
`160`	`163`	`public String getDisplayName() {`
`161`	`164`	`return GitHubCommitNotifier_DisplayName();`
`162`	`165`	`}`