Skip to content

Conversation

@vincbeck
Copy link
Contributor

@vincbeck vincbeck commented Sep 18, 2025

Current version of Fab provider is only compatible with Airflow 3. There is no need to check whether Airflow version is at least 2.9 in Fab provider. Actually in any provider because the minimum airflow version in providers is currently 2.10.

Note: With this change, ignore_provider_compatibility_error is never used (but mentioned in the doc). Not sure if we want to keep it?


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@vincbeck vincbeck force-pushed the vincbeck/fab_cleanup branch from 2aed2e5 to 9c4ddd7 Compare September 18, 2025 18:14
@vincbeck vincbeck changed the title Remove Airflow version compatibility checks in Fab provider Remove useless Airflow version compatibility checks Sep 18, 2025
@ashb ashb added the full tests needed We need to run full set of tests for this PR to merge label Sep 22, 2025
@ashb ashb closed this Sep 22, 2025
@ashb ashb reopened this Sep 22, 2025
@vincbeck vincbeck merged commit 8a6ecc5 into apache:main Sep 23, 2025
215 checks passed
@vincbeck vincbeck deleted the vincbeck/fab_cleanup branch September 23, 2025 15:28
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Sep 30, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 1, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 2, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 3, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 4, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 5, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 5, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 7, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 8, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 9, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 10, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 11, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 12, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 14, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 15, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 17, 2025
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers full tests needed We need to run full set of tests for this PR to merge kind:documentation provider:fab

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants