You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/09/30 17:14:35 UTC
[GitHub] [airflow] Jorricks opened a new pull request #18644: Document changes related to PR #16634
Jorricks opened a new pull request #18644:
URL: https://github.com/apache/airflow/pull/18644
As part of [16634](https://github.com/apache/airflow/pull/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 :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on pull request #18644: Document changes related to PR #16634
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #18644:
URL: https://github.com/apache/airflow/pull/18644#issuecomment-943242857
We are likely to release 2.2.1 shortly anyway so we can include it there.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk merged pull request #18644: Document changes related to PR #16634
Posted by GitBox <gi...@apache.org>.
potiuk merged pull request #18644:
URL: https://github.com/apache/airflow/pull/18644
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] Jorricks commented on pull request #18644: Document changes related to PR #16634
Posted by GitBox <gi...@apache.org>.
Jorricks commented on pull request #18644:
URL: https://github.com/apache/airflow/pull/18644#issuecomment-943235808
@kaxil @jhtimmens I think we should still merge this (even though we already released the Airflow 2.2.0 docs)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] github-actions[bot] commented on pull request #18644: Document changes related to PR #16634
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #18644:
URL: https://github.com/apache/airflow/pull/18644#issuecomment-931533820
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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org