Skip to content

Commit

Permalink
Resolve spotbugs error
Browse files Browse the repository at this point in the history
  • Loading branch information
yashpal2104 committed Jan 11, 2025
1 parent f3e735a commit 49af843
Showing 1 changed file with 3 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,8 @@
*/
package jenkins.advancedqueue.sorter;

import static org.jenkinsci.plugins.scriptsecurity.sandbox.groovy.GroovySandbox.LOGGER;

import edu.umd.cs.findbugs.annotations.CheckForNull;
import edu.umd.cs.findbugs.annotations.CheckReturnValue;
import edu.umd.cs.findbugs.annotations.NonNull;
Expand Down Expand Up @@ -70,10 +72,7 @@ public void onStartedItem(@NonNull LeftItem item, float weight) {}
@CheckReturnValue
public void onCanceledItem(@NonNull LeftItem item) {
if (item.isCancelled()) {

Check warning on line 74 in src/main/java/jenkins/advancedqueue/sorter/SorterStrategy.java

View check run for this annotation

ci.jenkins.io / Code Coverage

Partially covered line

Line 74 is only partially covered, one branch is missing
ExtensionList<SorterStrategy> all = all();
for (SorterStrategy prioritySorterStrategy : all) {
SorterStrategyDescriptor descriptor = prioritySorterStrategy.getDescriptor();
}
LOGGER.info("Item is cancelled: " + item);
}
}
;
Expand Down

0 comments on commit 49af843

Please sign in to comment.