Address another TODO

Signed-off-by: Thomas Newton <thomas.w.newton@gmail.com>

flytepropeller/pkg/controller/nodes/branch/evaluator.go

@@ -129,7 +129,8 @@ func DecideBranch(ctx context.Context, nl executors.NodeLookup, nodeID v1alpha1.
 		}
 		nStatus := nl.GetNodeExecutionStatus(ctx, n.GetID())
 		logger.Infof(ctx, "Branch Setting Node[%v] status to Skipped!", skippedNodeID)
-		// TODO: What is the impact of always setting clearStateOnAnyTermination=false here? 
+		// We hard code clearStateOnAnyTermination=false because it has no effect when setting phase to 
+		// NodePhaseSkipped. This saves us passing the config all the way down from the nodeExecutor
 		nStatus.UpdatePhase(v1alpha1.NodePhaseSkipped, v1.Now(), "Branch evaluated to false", false, nil)
 	}