potiuk commented on PR #25752: URL: https://github.com/apache/airflow/pull/25752#issuecomment-1222512339
> > A documentation should be added somewhere (docstring? TaskFlow howto?) to describe what returning True and False means. > > That's fair. I think updating the docs somewhere would be my preference. Maybe in the [Python Operator how-tos doc](https://airflow.apache.org/docs/apache-airflow/stable/howto/operator/python.html#howto-operator-shortcircuitoperator)? Sounds good. > Side note, maybe we should add a TaskFlow Decorators section to the [Python API Reference](https://airflow.apache.org/docs/apache-airflow/stable/python-api-ref.html) in a separate PR. WDYT? Agree. Also - since some of the decorators are coming from the providers, we should add a section in https://airflow.apache.org/docs/apache-airflow-providers/core-extensions/index.html (I can add it and also refactor this page a bit as it is a bit too long now and I think we should make it more compact). How about also reversing (in separate PR) the "priority" of "classic" vs. "taskflow" approach ? I think we are quickly approaching the point where taskflow is generally recommended and "on-par" approach than Classic and we should promote it as "the" way of doing things while marking classic approach as somewhat (not entirely but somewhat) legacty. -- 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]
