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 2020/10/06 09:46:42 UTC

[GitHub] [airflow] mik-laj commented on pull request #11277: Move dagrun_operator.py to dagrun.py (#11178)

mik-laj commented on pull request #11277:
URL: https://github.com/apache/airflow/pull/11277#issuecomment-704157344


   > I just noticed. In tests it is named as test_trigger_dagrun_operator_conf and test_trigger_dagrun_operator_templated_conf. Should those be changed to test_trigger_dagrun_conf and test_trigger_dagrun_templated_conf ?
   
   This is not necessary in my opinion. 
   
   > Move dagrun_operator.py to dagrun.py  
   
   I have doubts if this is a good name for a module. Wouldn't `trigger_dagrun` be better? @turbaszek WDYT


----------------------------------------------------------------
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:
users@infra.apache.org