ashb commented on code in PR #48683: URL: https://github.com/apache/airflow/pull/48683#discussion_r2028532770
########## providers/standard/tests/unit/standard/decorators/test_python_virtualenv.py: ########## @@ -41,6 +43,10 @@ _Invalid = Any [email protected]( + not AIRFLOW_V_3_0_PLUS, + reason="Decorators were part of core not providers, so this test doesnt make sense for < AF3.", +) Review Comment: Just want to highlight this for others. The reason we did this, is that On Airflow 2.x, `task.python` and all the other previous "core" operators were hard-coded in https://github.com/apache/airflow/blob/2.10.5/airflow/decorators/__init__.py#L61-L71 Which means that it is impossible for users to use these decorators on Airflow 2, meaning we should skip these tests in the compat tests, as the decorators specifically won't ever be using the one from the standard provider, even if installed. -- 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]
