ashb commented on code in PR #51829:
URL: https://github.com/apache/airflow/pull/51829#discussion_r2151912185
##########
devel-common/src/tests_common/pytest_plugin.py:
##########
@@ -42,14 +42,13 @@
from itsdangerous import URLSafeSerializer
from sqlalchemy.orm import Session
- from airflow.models.baseoperator import BaseOperator
from airflow.models.dag import DAG, ScheduleArg
from airflow.models.dagrun import DagRun, DagRunType
from airflow.models.taskinstance import TaskInstance
from airflow.providers.standard.operators.empty import EmptyOperator
from airflow.sdk import Context
from airflow.sdk.api.datamodels._generated import TaskInstanceState as
TIState
- from airflow.sdk.bases.operator import BaseOperator as TaskSDKBaseOperator
+ from airflow.sdk.bases.operator import BaseOperator, BaseOperator as
TaskSDKBaseOperator
Review Comment:
This looks odd -- why are we importing it twice under different names?
--
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]
