potiuk commented on PR #44956: URL: https://github.com/apache/airflow/pull/44956#issuecomment-2546331550
Also we should remove all conditional code with `AIRFLOW_V_2_9_PLUS`. I've added a point about ti in "bumping min version", > 4. Remove `AIRFLOW_V_2_X_PLUS` in all tests (review and update skipif and other conditional > behaviour and test_compat.py, where X is the TARGET version we change to. For example > when we update min Airflow version to 2.9.0, we should remove all references to AIRFLOW_V_2_9_PLUS > simply because "everything" in our tests is already 2.9.0+ and there is no need to exclude or > modify tests for earlier versions of Airflow. https://github.com/apache/airflow/blob/main/dev/README_RELEASE_PROVIDER_PACKAGES.md#bump-min-airflow-version-for-providers I can do it tomorrow if no-one does it before and if no-one does it before. -- 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]
