Skip to content

Commit

Permalink
[JENKINS-42000] ProjectObserver.complete should be allowed to throw I…
Browse files Browse the repository at this point in the history
…OException

- While technically a binary incompatibility, as all call sites except for OrganizationFolder already declare throwing IOException this should be safe
  • Loading branch information
stephenc committed Feb 14, 2017
1 parent 7e3ccd6 commit b4b7b67
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/main/java/jenkins/scm/api/SCMSourceObserver.java
Expand Up @@ -29,6 +29,7 @@
import edu.umd.cs.findbugs.annotations.Nullable;
import hudson.model.Item;
import hudson.model.TaskListener;
import java.io.IOException;
import java.util.Arrays;
import java.util.HashSet;
import java.util.Set;
Expand Down Expand Up @@ -151,9 +152,10 @@ public abstract void addAttribute(@NonNull String key, @Nullable Object value) t
* To be called when finished defining one project.
*
* @throws IllegalStateException may be thrown if called twice
* @throws IOException if processing of the final project could not be completed due to an {@link IOException}.
* @throws InterruptedException if processing of the final project was interrupted
*/
public abstract void complete() throws IllegalStateException, InterruptedException;
public abstract void complete() throws IllegalStateException, IOException, InterruptedException;

}

Expand Down

0 comments on commit b4b7b67

Please sign in to comment.