You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by ra...@apache.org on 2026/03/25 11:47:56 UTC

(airflow) 15/46: update check_clie_definition_import to allow using common provider sdk (#64138)

This is an automated email from the ASF dual-hosted git repository.

rahulvats pushed a commit to branch py-client-sync
in repository https://gitbox.apache.org/repos/asf/airflow.git

commit 39e5548f3c26e9c1f725e58d2579fb936293e8f2
Author: Jeongwoo Do <48...@users.noreply.github.com>
AuthorDate: Tue Mar 24 15:52:25 2026 +0900

    update check_clie_definition_import to allow using common provider sdk (#64138)
    
    * update check_clie_definition_import to allow using common provider sdk
    
    * fix allowed module
    
    * fix allowed module
---
 scripts/ci/prek/check_cli_definition_imports.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/scripts/ci/prek/check_cli_definition_imports.py b/scripts/ci/prek/check_cli_definition_imports.py
index c8a9d1730af..de3aa60f6e5 100755
--- a/scripts/ci/prek/check_cli_definition_imports.py
+++ b/scripts/ci/prek/check_cli_definition_imports.py
@@ -42,6 +42,7 @@ from common_prek_utils import console, get_imports_from_file
 ALLOWED_MODULES = {
     "airflow.configuration",
     "airflow.cli.cli_config",
+    "airflow.providers.common.compat.sdk.conf",
 }
 
 # Standard library and __future__ modules are also allowed
@@ -164,6 +165,7 @@ def main() -> int:
             f"[yellow]CLI definition files (*/cli/definition.py) should only import from:[/]\n"
             "  - airflow.configuration\n"
             "  - airflow.cli.cli_config\n"
+            "  - airflow.providers.common.compat.sdk.conf\n"
             "  - Their own provider's version_compat module\n"
             f"  - Standard library modules ({', '.join(STDLIB_PREFIXES)})\n"
         )