Vamsi-klu commented on code in PR #60435:
URL: https://github.com/apache/airflow/pull/60435#discussion_r2690935637


##########
airflow-core/src/airflow/serialization/definitions/baseoperator.py:
##########
@@ -141,7 +141,7 @@ class SerializedBaseOperator(DAGNode):
 
     trigger_rule: str | TriggerRule = "all_success"
 
-    # TODO: Remove the following, they aren't used anymore
+    # Used by dot_renderer.py for DAG graph visualization and exposed in the 
API
     ui_color: str = "#fff"
     ui_fgcolor: str = "#000"

Review Comment:
   Thanks for calling this out. I removed that change from this PR and tracked 
the removal in #60465 so it can be handled separately.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to