Skip to content

Commit

Permalink
More clearly noting when jenkinsci/jenkins#7599 was merged
Browse files Browse the repository at this point in the history
  • Loading branch information
jglick committed Feb 15, 2023
1 parent 0824931 commit 0a6bca5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -569,7 +569,7 @@ public String getShortDescription() {
return r;
}

// TODO https://github.com/jenkinsci/jenkins/pull/7599 @Override
// TODO 2.389+ @Override
public @CheckForNull Queue.Executable getOwnerExecutable() {
Run<?, ?> r = runForDisplay();
return r instanceof Queue.Executable ? (Queue.Executable) r : null;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -613,7 +613,7 @@ private static void assertLogMatches(WorkflowRun build, String regexp) throws IO
Queue.Item[] items = Queue.getInstance().getItems();
assertEquals(1, items.length);
assertEquals(p, items[0].task.getOwnerTask());
// // TODO https://github.com/jenkinsci/jenkins/pull/7599 remove cast
// // TODO 2.389+ remove cast
assertEquals(b, ((ExecutorStepExecution.PlaceholderTask) items[0].task).getOwnerExecutable());
assertEquals(items[0], QueueItemAction.getQueueItem(executorStartNode));

Expand Down

0 comments on commit 0a6bca5

Please sign in to comment.