uranusjr commented on code in PR #37028:
URL: https://github.com/apache/airflow/pull/37028#discussion_r1625654355
##########
airflow/models/baseoperator.py:
##########
@@ -1354,7 +1354,7 @@ def render_template_fields(
context: Context,
jinja_env: jinja2.Environment | None = None,
) -> None:
- """Template all attributes listed in *self.template_fields*.
+ """Template all attributes listed in *self.template_fields* except for
callable.
Review Comment:
I think we can just change this to say like **Render** all attributes listed
in *self.template_fields*? Since we do handle callables in this function (by
way of `_do_render_template_fields`, they’re just not templated.
--
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]
