You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/07/13 06:56:23 UTC

[GitHub] [airflow] lssatvik commented on a change in pull request #16958: Update tutorial.rst

lssatvik commented on a change in pull request #16958:
URL: https://github.com/apache/airflow/pull/16958#discussion_r668479594



##########
File path: docs/apache-airflow/tutorial.rst
##########
@@ -154,7 +154,7 @@ references parameters like ``{{ ds }}``, calls a function as in
 ``{{ macros.ds_add(ds, 7)}}``, and references a user-defined parameter
 in ``{{ params.my_param }}``.
 
-The ``params`` hook in ``BaseOperator`` allows you to pass a dictionary of
+The ``params`` hook in ``BashOperator`` allows you to pass a dictionary of

Review comment:
       Not saying it isn't correct, but that while explaining the arguments of BashOperator, why suddenly refer to BaseOperator? It improves understanding if you refer to BashOperator instead which is also correct. The paragraph is explaining BashOperator rather than BaseOperator. 
   Any special purpose in referring to BaseOperator instead of BashOperator there?




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