potiuk commented on a change in pull request #6597: [AIRFLOW-6005] Common 
classes should always be imported from airflow, Depends on [AIRFLOW-6128]
URL: https://github.com/apache/airflow/pull/6597#discussion_r378269370
 
 

 ##########
 File path: .pre-commit-config.yaml
 ##########
 @@ -237,6 +237,43 @@ repos:
           ^airflow/sensors/.*$|
           ^airflow/providers/.*\.py$|
           ^airflow/contrib/.*\.py$
+      - id: common-imports
+        language: pygrep
+        name: Make sure LoggingMixin is imported consistently 'from airflow'
+        entry: "from airflow\\.utils\\.log\\.logging_mixin import.* 
LoggingMixin"
+        pass_filenames: true
+        exclude: >
+          (?x)
+          ^airflow/__init__\.py$|
+          ^airflow/plugins_manager.py$|
+          ^airflow/configuration.py$|
+          ^airflow/utils/sqlalchemy.py$
+        files: \.py$
+      - id: common-imports
+        language: pygrep
+        name: Make sure conf is imported consistently 'from airflow'
+        entry: "from airflow\\.configuration import.* conf"
+        pass_filenames: true
+        exclude: >
+          (?x)
+          ^airflow/__init__\.py$|
+          ^airflow/logging_config.py$|
+          ^airflow/settings.py$
+        files: \.py$
+      - id: common-imports
+        language: pygrep
+        name: Make sure AirflowConfigException is imported consistently 'from 
airflow'
+        entry: "from airflow\\.exception import.* AirflowConfigException"
 
 Review comment:
   Will come back to it some day :).

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


With regards,
Apache Git Services

Reply via email to