[GitHub] [airflow] tirkarthi commented on a diff in pull request #28256: Include full path to Python files under zip path while clearing import errors.

2023-02-21 Thread via GitHub
tirkarthi commented on code in PR #28256: URL: https://github.com/apache/airflow/pull/28256#discussion_r1112980514 ## airflow/dag_processing/manager.py: ## @@ -782,7 +782,11 @@ def clear_nonexistent_import_errors(file_paths: list[str] | None, session=NEW_SE """

[GitHub] [airflow] tirkarthi commented on a diff in pull request #28256: Include full path to Python files under zip path while clearing import errors.

2023-02-20 Thread via GitHub
tirkarthi commented on code in PR #28256: URL: https://github.com/apache/airflow/pull/28256#discussion_r1112035762 ## airflow/dag_processing/manager.py: ## @@ -782,7 +782,11 @@ def clear_nonexistent_import_errors(file_paths: list[str] | None, session=NEW_SE """

[GitHub] [airflow] tirkarthi commented on a diff in pull request #28256: Include full path to Python files under zip path while clearing import errors.

2023-02-20 Thread via GitHub
tirkarthi commented on code in PR #28256: URL: https://github.com/apache/airflow/pull/28256#discussion_r644805 ## airflow/dag_processing/manager.py: ## @@ -782,7 +782,11 @@ def clear_nonexistent_import_errors(file_paths: list[str] | None, session=NEW_SE """

[GitHub] [airflow] tirkarthi commented on a diff in pull request #28256: Include full path to Python files under zip path while clearing import errors.

2023-01-22 Thread via GitHub
tirkarthi commented on code in PR #28256: URL: https://github.com/apache/airflow/pull/28256#discussion_r1083451283 ## airflow/utils/file.py: ## @@ -324,6 +326,12 @@ def find_dag_file_paths(directory: str | pathlib.Path, safe_mode: bool) -> list[ continue