fredthomsen commented on code in PR #69426:
URL: https://github.com/apache/airflow/pull/69426#discussion_r3670936172


##########
airflow-core/docs/templates-ref.rst:
##########
@@ -107,6 +107,7 @@ Variable                                    Type            
      Description
 ``{{ expanded_ti_count }}``                 int | ``None``        | Number of 
task instances that a mapped task was expanded into. If
                                                                   | the 
current task is not mapped, this should be ``None``.
                                                                   | Added in 
version 2.5.
+``{{ extra_links }}``                       ExtraLinksAccessor    Accessor for 
operator extra links defined on the task. Added in version 3.4.

Review Comment:
   Agree, this is a pretty generic parameter name and could result in breakage. 
 Would making it `task_extra_links` reduce the chances of collisions enough or 
are there other suggestions?  Given other new context keys have been added 
within a major release before has there been guidance around best approach?



-- 
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