uranusjr commented on code in PR #59817: URL: https://github.com/apache/airflow/pull/59817#discussion_r2648308179
########## providers/imap/src/airflow/providers/imap/sensors/imap_attachment.py: ########## @@ -26,11 +26,7 @@ from airflow.providers.imap.hooks.imap import ImapHook if TYPE_CHECKING: - try: - from airflow.sdk.definitions.context import Context - except ImportError: - # TODO: Remove once provider drops support for Airflow 2 - from airflow.utils.context import Context + from airflow.sdk import Context Review Comment: This is for typing. Even if you’re on Airflow 2, the try-except pattern would not work. It is useless. -- 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]
