Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds updating note to .airflowignore fix #12043

Merged

Conversation

potiuk
Copy link
Member

@potiuk potiuk commented Nov 2, 2020

This is a follow-up after #11993 - the behaviour of airflowignore
changes significantly enough to require a warning.


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, 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 UPDATING.md.

This is a follow-up after apache#11993 - the behaviour of airflowignore
changes significantly enough to require a warning.
@potiuk potiuk requested a review from ashb November 2, 2020 14:56
@potiuk
Copy link
Member Author

potiuk commented Nov 2, 2020

cc: @megaserg

@github-actions github-actions bot added the okay to merge It's ok to merge this PR as it does not require more tests label Nov 2, 2020
@github-actions
Copy link

github-actions bot commented Nov 2, 2020

The PR is ready to be merged. No tests are needed!

@kaxil kaxil changed the title Adds updating note to ariflowignore fix Adds updating note to airflowignore fix Nov 2, 2020
@kaxil kaxil changed the title Adds updating note to airflowignore fix Adds updating note to .airflowignore fix Nov 2, 2020
@potiuk potiuk merged commit b72bd4a into apache:master Nov 2, 2020
@potiuk potiuk deleted the aadd-updating-note-for-airflowignore branch November 2, 2020 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
okay to merge It's ok to merge this PR as it does not require more tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants