casassg commented on a change in pull request #12271:
URL: https://github.com/apache/airflow/pull/12271#discussion_r520953070
##########
File path: airflow/example_dags/tutorial_taskflow_api_etl.py
##########
@@ -33,6 +33,7 @@
# The DAG object; we'll need this to instantiate a DAG
from airflow import DAG
Review comment:
This is no longer needed!
```suggestion
```
##########
File path: airflow/example_dags/tutorial_taskflow_api_etl.py
##########
@@ -45,14 +46,8 @@
# [END default_args]
# [START instantiate_dag]
-with DAG(
- 'tutorial_taskflow_api_etl_dag',
- default_args=default_args,
- description='TaskFlow API ETL DAG tutorial',
- schedule_interval=None,
- start_date=days_ago(2),
- tags=['example'],
-) as dag:
+@dag(default_args=default_args, schedule_interval=None, start_date=days_ago(2))
+def tutorial_taskflow_api_etl():
Review comment:
Documentation in `@dag` decorator should be a function `__doc__`
```suggestion
def tutorial_taskflow_api_etl():
"""### TaskFlow API Tutorial Documentation
This is a simple ETL data pipeline example which demonstrates the use of
the TaskFlow API
using three simple tasks for Extract, Transform, and Load.
Documentation that goes along with the Airflow TaskFlow API tutorial
located
[here](https://airflow.apache.org/docs/stable/tutorial_taskflow_api.html)
"""
```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
