Add missing airflow config lint rules and update/fix newsfragments#49048
Merged
Lee-W merged 5 commits intoapache:mainfrom Apr 10, 2025
Merged
Add missing airflow config lint rules and update/fix newsfragments#49048Lee-W merged 5 commits intoapache:mainfrom
airflow config lint rules and update/fix newsfragments#49048Lee-W merged 5 commits intoapache:mainfrom
Conversation
Member
Author
|
there're still some rules not implemented but I'm out of bandwidth to work on them at this moment |
kaxil
approved these changes
Apr 10, 2025
…nificant.rst reading apache#48528, it suggest users to use dag.test() instead of replacing the value as dag.test()
…heduler.create_delta_data_intervals to config lint
…s and scheduler.create_delta_data_intervals to config lint" This reverts commit e60bfe3.
Lee-W
force-pushed
the
update-news-fragments
branch
from
e60bfe3 to
640daa1
Compare
Member
Author
|
Revert the following rules as they're failing and I'm out of bandwidth to take a deeper look
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Some of the
airflow config lintrules was not implemented back to the time changes were made. Also found some outdated newsfragmentsWhat
airflow config lintrules^ 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.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.