amoghrajesh commented on code in PR #46231:
URL: https://github.com/apache/airflow/pull/46231#discussion_r1936929742
##########
tests_common/pytest_plugin.py:
##########
@@ -40,7 +40,7 @@
from airflow.models.dag import DAG, ScheduleArg
from airflow.models.dagrun import DagRun, DagRunType
from airflow.models.taskinstance import TaskInstance
- from airflow.operators.empty import EmptyOperator
+ from airflow.providers.standard.operators.empty import EmptyOperator
Review Comment:
Thats because it is in a type checking block used for mypy etc. But we
import it normally in functions that need it for sake of usage
##########
tests/serialization/test_dag_serialization.py:
##########
@@ -64,9 +64,9 @@
from airflow.models.expandinput import EXPAND_INPUT_EMPTY
from airflow.models.mappedoperator import MappedOperator
from airflow.models.xcom import XCom
-from airflow.operators.empty import EmptyOperator
from airflow.providers.cncf.kubernetes.pod_generator import PodGenerator
from airflow.providers.standard.operators.bash import BashOperator
+from airflow.providers.standard.operators.empty import EmptyOperator
Review Comment:
Thats because it is in a type checking block used for mypy etc. But we
import it normally in functions that need it for sake of usage
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
