Skip to content

Conversation

@XD-DENG
Copy link
Member

@XD-DENG XD-DENG commented Nov 24, 2020


^ 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.

@XD-DENG
Copy link
Member Author

XD-DENG commented Nov 24, 2020

CC @ashb . If we have b4 for 2.0.0, this doc fix should be included, given it's fixing for #9477 which is included in 2.0.0a1.

Copy link
Member

@kaxil kaxil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch

@github-actions
Copy link

The PR needs to run all tests because it modifies core of Airflow! Please rebase it to latest master or ask committer to re-run it!

@github-actions github-actions bot added the full tests needed We need to run full set of tests for this PR to merge label Nov 24, 2020
@XD-DENG XD-DENG removed the full tests needed We need to run full set of tests for this PR to merge label Nov 24, 2020
- Support towards YAML is added in PR apache#9477
  Most docs were updated for this. But a few docstrings and exception logging were missed

- Fix a minor error in the doc .rst file.
@XD-DENG XD-DENG force-pushed the localfilesystem-secrets-backend branch from 12d599d to f22e98b Compare November 24, 2020 19:38
@ashb ashb added this to the Airflow 2.0.0-beta4 milestone Nov 24, 2020
@kaxil kaxil merged commit c6467ba into apache:master Nov 25, 2020
@XD-DENG XD-DENG deleted the localfilesystem-secrets-backend branch March 26, 2021 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants