Skip to content

Adds updating note to .airflowignore fix#12043

Merged
potiuk merged 1 commit into
apache:masterfrom
PolideaInternal:aadd-updating-note-for-airflowignore
Nov 2, 2020
Merged

Adds updating note to .airflowignore fix#12043
potiuk merged 1 commit into
apache:masterfrom
PolideaInternal:aadd-updating-note-for-airflowignore

Conversation

@potiuk

@potiuk potiuk commented Nov 2, 2020

Copy link
Copy Markdown
Member

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

potiuk commented Nov 2, 2020

Copy link
Copy Markdown
Member Author

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

github-actions Bot commented Nov 2, 2020

Copy link
Copy Markdown
Contributor

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