You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "AzisK (via GitHub)" <gi...@apache.org> on 2025/05/09 16:03:23 UTC
[PR] Correct PythonOperator reference from the first tutorial to BashOpera… [airflow]
AzisK opened a new pull request, #50404:
URL: https://github.com/apache/airflow/pull/50404
…tor in taskflow.rst
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Correct PythonOperator reference from the first tutorial to BashOpera… [airflow]
Posted by "AzisK (via GitHub)" <gi...@apache.org>.
AzisK commented on code in PR #50404:
URL: https://github.com/apache/airflow/pull/50404#discussion_r2084323038
##########
airflow-core/docs/tutorial/taskflow.rst:
##########
@@ -19,7 +19,7 @@
Pythonic DAGs with the TaskFlow API
===================================
-In the first tutorial, you built your first Airflow DAG using traditional Operators like ``PythonOperator``.
+In the first tutorial, you built your first Airflow DAG using traditional Operators like ``BashOperator``.
Review Comment:
The first tutorial has no usage of `PythonOperator` https://airflow.apache.org/docs/apache-airflow/stable/tutorial/fundamentals.html
We can solve this in 2 ways
1) Currently offered solution saying that we built the first tutorial using `BashOperator`s
2) Build a second tutorial which would use `PythonOperator`s and then "Pythonic DAGs with the TaskFlow API" would be the third tutorial
2nd way is more work but I could work on it if you agree? @kaxil
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Correct PythonOperator reference from the first tutorial to BashOpera… [airflow]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #50404:
URL: https://github.com/apache/airflow/pull/50404
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Correct PythonOperator reference from the first tutorial to BashOpera… [airflow]
Posted by "kaxil (via GitHub)" <gi...@apache.org>.
kaxil commented on code in PR #50404:
URL: https://github.com/apache/airflow/pull/50404#discussion_r2084368600
##########
airflow-core/docs/tutorial/taskflow.rst:
##########
@@ -19,7 +19,7 @@
Pythonic DAGs with the TaskFlow API
===================================
-In the first tutorial, you built your first Airflow DAG using traditional Operators like ``PythonOperator``.
+In the first tutorial, you built your first Airflow DAG using traditional Operators like ``BashOperator``.
Review Comment:
aah makes sense, all good 👍
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Correct PythonOperator reference from the first tutorial to BashOpera… [airflow]
Posted by "kaxil (via GitHub)" <gi...@apache.org>.
kaxil commented on code in PR #50404:
URL: https://github.com/apache/airflow/pull/50404#discussion_r2084272278
##########
airflow-core/docs/tutorial/taskflow.rst:
##########
@@ -19,7 +19,7 @@
Pythonic DAGs with the TaskFlow API
===================================
-In the first tutorial, you built your first Airflow DAG using traditional Operators like ``PythonOperator``.
+In the first tutorial, you built your first Airflow DAG using traditional Operators like ``BashOperator``.
Review Comment:
Why was this changed? What was wrong? @AzisK ?
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Correct PythonOperator reference from the first tutorial to BashOpera… [airflow]
Posted by "AzisK (via GitHub)" <gi...@apache.org>.
AzisK commented on PR #50404:
URL: https://github.com/apache/airflow/pull/50404#issuecomment-2869777116
Hi @potiuk. I am tagging you because I have seen you approve other pull requests. This is a simple documentation correction when you have a chance
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org