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

Document changes related to PR #16634 #18644

Merged
merged 1 commit into from
Oct 14, 2021

Conversation

Jorricks
Copy link
Contributor

As part of 16634, we also should have some documentation changes.
I was not really sure whether we should do:

  1. A comparison between the permissions in Airflow 2.{0,1} and Airflow 2 in the table
  2. A small message at the top indicating the permissions changed with Airflow 2.2.

Let me know what you think :)

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.

@github-actions
Copy link

The PR is likely ready to be merged. No tests are needed as no important environment files, nor python files were modified by it. However, committers might decide that full test matrix is needed and add the 'full tests needed' label. Then you should rebase it to the latest main or amend the last commit of the PR, and push it with --force-with-lease.

@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 Sep 30, 2021
@Jorricks
Copy link
Contributor Author

@kaxil @jhtimmens I think we should still merge this (even though we already released the Airflow 2.2.0 docs)

@potiuk potiuk added this to the Airflow 2.2.1 milestone Oct 14, 2021
@potiuk
Copy link
Member

potiuk commented Oct 14, 2021

We are likely to release 2.2.1 shortly anyway so we can include it there.

@potiuk potiuk merged commit eb2c7af into apache:main Oct 14, 2021
jedcunningham pushed a commit that referenced this pull request Oct 14, 2021
(cherry picked from commit eb2c7af)
@jedcunningham jedcunningham added the type:doc-only Changelog: Doc Only label Apr 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:documentation okay to merge It's ok to merge this PR as it does not require more tests type:doc-only Changelog: Doc Only
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants