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

kaxilnaik pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new e778f033bf Fix broken main with static checks (#43287)
e778f033bf is described below

commit e778f033bfd677e4b6adbb144f0f169141d92331
Author: Kaxil Naik <[email protected]>
AuthorDate: Wed Oct 23 01:39:08 2024 +0100

    Fix broken main with static checks (#43287)
    
    Change in https://github.com/apache/airflow/pull/43281 change the python 
exec path.
    
    ```
    ❯ ipython
    Python 3.11.10 (main, Sep  9 2024, 20:58:40) [Clang 18.1.8 ]
    Type 'copyright', 'credits' or 'license' for more information
    IPython 8.18.1 -- An enhanced Interactive Python. Type '?' for help.
    
    In [1]: import sys
    
    In [2]: sys.executable[0]
    Out[2]: '/'
    
    In [3]: sys.executable[1]
    Out[3]: 'U'
    
    In [4]: sys.executable
    Out[4]: 
'/Users/kaxilnaik/Documents/GitHub/astronomer/airflow/.venv/bin/python3'
    ```
    
    Example Failure:
    
https://github.com/apache/airflow/actions/runs/11470387014/job/31919681910?pr=43286
---
 scripts/tools/check_if_limited_dependencies.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/tools/check_if_limited_dependencies.py 
b/scripts/tools/check_if_limited_dependencies.py
index 7ce617a322..03f513f256 100755
--- a/scripts/tools/check_if_limited_dependencies.py
+++ b/scripts/tools/check_if_limited_dependencies.py
@@ -43,7 +43,7 @@ console = Console(
 env = os.environ.copy()
 env["PYTHONPATH"] = str(DEV_BREEZE_SRC_PATH)
 try:
-    check_call([sys.executable[0], DEV_BREEZE_SRC_PATH / "airflow_breeze" / 
"breeze.py", "--help"], env=env)
+    check_call([sys.executable, DEV_BREEZE_SRC_PATH / "airflow_breeze" / 
"breeze.py", "--help"], env=env)
 except CalledProcessError:
     console.print("[red]Breeze should only use limited dependencies when 
imported (see errors above).[/]\n")
     console.print(

Reply via email to