Commit
Add check for non-AbstractBuild stuff
- Loading branch information
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,7 +4,7 @@ | |
<parent> | ||
<groupId>org.jenkins-ci.plugins</groupId> | ||
<artifactId>plugin</artifactId> | ||
<version>1.580.1</version> | ||
<version>1.609.2</version> | ||
This comment has been minimized.
Sorry, something went wrong.
This comment has been minimized.
Sorry, something went wrong.
slide
Author
Member
|
||
</parent> | ||
|
||
<licenses> | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -80,15 +80,15 @@ public void send(final String format, final Object... args) { | |
} else { | ||
users = new HashSet<User>(); | ||
debug.send("Collecting builds with suspects..."); | ||
final HashSet<AbstractBuild<?, ?>> buildsWithSuspects = new HashSet<AbstractBuild<?, ?>>(); | ||
final HashSet<Run<?, ?>> buildsWithSuspects = new HashSet<Run<?, ?>>(); | ||
Run<?, ?> firstFailedBuild = currentBuild; | ||
Run<?, ?> candidate = currentBuild; | ||
while (candidate != null && candidate.getResult().isWorseOrEqualTo(Result.FAILURE)) { | ||
firstFailedBuild = candidate; | ||
candidate = candidate.getPreviousCompletedBuild(); | ||
} | ||
if (firstFailedBuild instanceof AbstractBuild) { | ||
This comment has been minimized.
Sorry, something went wrong.
vtintillier
Member
|
||
buildsWithSuspects.add((AbstractBuild<?, ?>) firstFailedBuild); | ||
buildsWithSuspects.add(firstFailedBuild); | ||
} else { | ||
debug.send(" firstFailedBuild was not an instance of AbstractBuild"); | ||
} | ||
|
Out of curiosity, was this core update needed for some new API?