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

add some override annotations in ghcommitnotifier
jenkinsci · Jul 21, 2016 · 6fdefd9 · 6fdefd9
2 parents 6b10f5a + 9649beb
commit 6fdefd9
Showing 1 changed file with 3 additions and 0 deletions.
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();
         }