Deleting comment; for now it seems clearest to keep `ExecutorStepDyna…

…micContext` as a distinct type
jglick · Jan 5, 2022 · 3f0e140 · 3f0e140
1 parent e1b4bb7
commit 3f0e140
Showing 1 changed file with 0 additions and 1 deletion.
diff --git a/src/main/java/org/jenkinsci/plugins/workflow/support/steps/ExecutorStepExecution.java b/src/main/java/org/jenkinsci/plugins/workflow/support/steps/ExecutorStepExecution.java
@@ -91,7 +91,6 @@ public class ExecutorStepExecution extends AbstractStepExecutionImpl {
     public static long TIMEOUT_WAITING_FOR_NODE_MILLIS = Main.isUnitTest ? /* fail faster */ TimeUnit.SECONDS.toMillis(15) : Long.getLong("org.jenkinsci.plugins.workflow.support.pickles.ExecutorPickle.timeoutForNodeMillis", TimeUnit.MINUTES.toMillis(5));
 
     private final ExecutorStep step;
-    // TODO perhaps just inline it here? does not do much good as a separate class
     private ExecutorStepDynamicContext state;
 
     ExecutorStepExecution(StepContext context, ExecutorStep step) {