ashb commented on code in PR #54505: URL: https://github.com/apache/airflow/pull/54505#discussion_r2559466169
########## airflow-core/tests/unit/dags/test_assets.py: ########## @@ -19,11 +19,11 @@ from datetime import datetime -from airflow.exceptions import AirflowFailException, AirflowSkipException from airflow.models.dag import DAG from airflow.providers.standard.operators.bash import BashOperator from airflow.providers.standard.operators.python import PythonOperator from airflow.sdk.definitions.asset import Asset +from airflow.sdk.exceptions import AirflowFailException, AirflowSkipException Review Comment: If this test is importing this and needing it is probably a sign that this some part of test shouldn't be in airflow-core. I.d say that we _also_ shouldn't be knowingly having code in Tests that issues deprecation warnings. -- 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]
