kaxil commented on code in PR #45627: URL: https://github.com/apache/airflow/pull/45627#discussion_r1914826405
########## airflow/models/abstractoperator.py: ########## @@ -19,40 +19,35 @@ import datetime import inspect -from collections.abc import Iterable, Iterator, Mapping, Sequence +from collections.abc import Iterable, Sequence from functools import cached_property from typing import TYPE_CHECKING, Any, Callable -import methodtools from sqlalchemy import select from airflow.configuration import conf from airflow.exceptions import AirflowException from airflow.models.expandinput import NotFullyPopulated -from airflow.sdk.definitions._internal.abstractoperator import AbstractOperator as TaskSDKAbstractOperator +from airflow.sdk.definitions._internal.abstractoperator import ( + AbstractOperator as TaskSDKAbstractOperator, + NotMapped as NotMapped, Review Comment: and https://docs.astral.sh/ruff/rules/unused-import/ ########## airflow/models/abstractoperator.py: ########## @@ -19,40 +19,35 @@ import datetime import inspect -from collections.abc import Iterable, Iterator, Mapping, Sequence +from collections.abc import Iterable, Sequence from functools import cached_property from typing import TYPE_CHECKING, Any, Callable -import methodtools from sqlalchemy import select from airflow.configuration import conf from airflow.exceptions import AirflowException from airflow.models.expandinput import NotFullyPopulated -from airflow.sdk.definitions._internal.abstractoperator import AbstractOperator as TaskSDKAbstractOperator +from airflow.sdk.definitions._internal.abstractoperator import ( + AbstractOperator as TaskSDKAbstractOperator, + NotMapped as NotMapped, Review Comment: and https://docs.astral.sh/ruff/rules/unused-import/ >If an import statement is used to re-export a symbol as part of a module's public interface, consider using a "redundant" import alias, which instructs Ruff (and other tools) to respect the re-export, and avoid marking it as unused, as in: -- 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]
