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/09/03 21:39:53 UTC

[GitHub] [airflow] potiuk commented on pull request #18019: Adding missing init file in Alibaba example_dags directory

potiuk commented on pull request #18019:
URL: https://github.com/apache/airflow/pull/18019#issuecomment-912825048


   No problem. It's a minor problem and indeed missed by our tests - adding a CI test to verify that sounds like a cool idea. We already have a check that `providers/__init_.py` should not exist (it's the only implict namespace package we have due to the way we handled backport providers): https://github.com/apache/airflow/blob/main/scripts/ci/pre_commit/pre_commit_check_providers_init.sh 
   
   So maybe it can be added as pre-commit in similar way.


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