diff --git a/flytepropeller/pkg/controller/nodes/branch/evaluator.go b/flytepropeller/pkg/controller/nodes/branch/evaluator.go
index 9d4aede5e9..ee58570d97 100644
--- a/flytepropeller/pkg/controller/nodes/branch/evaluator.go
+++ b/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)
 	}