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/07/07 19:41:25 UTC
[GitHub] [airflow] jhtimmins opened a new pull request #16862: Sync v2 1 2 stable v2
jhtimmins opened a new pull request #16862:
URL: https://github.com/apache/airflow/pull/16862
Sync remaining commits into v2-1-2
--
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 #16862: Sync v2 1 2 stable v2
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #16862:
URL: https://github.com/apache/airflow/pull/16862#issuecomment-875903554
i think cherry picking 3db347edcfe444a67e59e8cf0019e80a02dbceab and 866a601b76e219b3c043e1dbbc8fb22300866351 should bring the compat change and remove pylint (some more cherry-picks might be needed)
--
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] jhtimmins merged pull request #16862: Sync v2 1 2 stable v2
Posted by GitBox <gi...@apache.org>.
jhtimmins merged pull request #16862:
URL: https://github.com/apache/airflow/pull/16862
--
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 #16862: Sync v2 1 2 stable v2
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #16862:
URL: https://github.com/apache/airflow/pull/16862#issuecomment-875888462
The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest main at your convenience, 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
[GitHub] [airflow] potiuk commented on pull request #16862: Sync v2 1 2 stable v2
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #16862:
URL: https://github.com/apache/airflow/pull/16862#issuecomment-875894878
BTW. Something wrong with airflow.compat missing (both pylint and tests)
--
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 #16862: Sync v2 1 2 stable v2
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #16862:
URL: https://github.com/apache/airflow/pull/16862#issuecomment-875900026
Seems like this one https://github.com/apache/airflow/pull/16700 brings airflow.compat ?
--
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 #16862: Sync v2 1 2 stable v2
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #16862:
URL: https://github.com/apache/airflow/pull/16862#issuecomment-875935096
BTW. Constraints upgraded for `constraints-2-1` branch - following https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#manually-generating-constraint-files
--
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 #16862: Sync v2 1 2 stable v2
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #16862:
URL: https://github.com/apache/airflow/pull/16862#issuecomment-875889063
Might also make sense to run constraints upgrade. I will run it
--
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] jhtimmins closed pull request #16862: Sync v2 1 2 stable v2
Posted by GitBox <gi...@apache.org>.
jhtimmins closed pull request #16862:
URL: https://github.com/apache/airflow/pull/16862
--
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