[GitHub] [airflow] potiuk 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
potiuk commented on code in PR #28256: URL: https://github.com/apache/airflow/pull/28256#discussion_r1113025147 ## airflow/dag_processing/manager.py: ## @@ -782,7 +782,11 @@ def clear_nonexistent_import_errors(file_paths: list[str] | None, session=NEW_SE """ q

[GitHub] [airflow] potiuk 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
potiuk commented on code in PR #28256: URL: https://github.com/apache/airflow/pull/28256#discussion_r1113020259 ## airflow/dag_processing/manager.py: ## @@ -782,7 +782,11 @@ def clear_nonexistent_import_errors(file_paths: list[str] | None, session=NEW_SE """ q

[GitHub] [airflow] potiuk 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
potiuk commented on code in PR #28256: URL: https://github.com/apache/airflow/pull/28256#discussion_r1112190405 ## airflow/dag_processing/manager.py: ## @@ -782,7 +782,11 @@ def clear_nonexistent_import_errors(file_paths: list[str] | None, session=NEW_SE """ q

[GitHub] [airflow] potiuk 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
potiuk commented on code in PR #28256: URL: https://github.com/apache/airflow/pull/28256#discussion_r1112190405 ## airflow/dag_processing/manager.py: ## @@ -782,7 +782,11 @@ def clear_nonexistent_import_errors(file_paths: list[str] | None, session=NEW_SE """ q

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

2022-12-27 Thread GitBox
potiuk commented on code in PR #28256: URL: https://github.com/apache/airflow/pull/28256#discussion_r1058096682 ## airflow/dag_processing/manager.py: ## @@ -777,8 +777,9 @@ def clear_nonexistent_import_errors(self, session): :param session: session for ORM operations

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

2022-12-27 Thread GitBox
potiuk commented on code in PR #28256: URL: https://github.com/apache/airflow/pull/28256#discussion_r1057903720 ## airflow/dag_processing/manager.py: ## @@ -777,8 +777,9 @@ def clear_nonexistent_import_errors(self, session): :param session: session for ORM operations