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 2020/08/03 22:34:59 UTC
[GitHub] [airflow] potiuk opened a new pull request #10143: Bring back coverage
potiuk opened a new pull request #10143:
URL: https://github.com/apache/airflow/pull/10143
Fixes #10138
---
**^ Add meaningful description above**
Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)** for more information.
In case of fundamental code change, Airflow Improvement Proposal ([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvements+Proposals)) is needed.
In case of a new dependency, check compliance with the [ASF 3rd Party License Policy](https://www.apache.org/legal/resolved.html#category-x).
In case of backwards incompatible changes please leave a note in [UPDATING.md](https://github.com/apache/airflow/blob/master/UPDATING.md).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on pull request #10143: Bring back coverage
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #10143:
URL: https://github.com/apache/airflow/pull/10143#issuecomment-668735194
Yep. This one brings coverage Back: https://codecov.io/gh/apache/airflow/commit/304b17532b0672a4947df74464cd8d7a8c68bf3e - we are at 89%
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk merged pull request #10143: Bring back coverage
Posted by GitBox <gi...@apache.org>.
potiuk merged pull request #10143:
URL: https://github.com/apache/airflow/pull/10143
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on pull request #10143: Bring back coverage
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #10143:
URL: https://github.com/apache/airflow/pull/10143#issuecomment-668699237
Looks like we are going to have code coverage back. When I tried to test the action from codecov it was broken yesterday (but it was really a github problem with spelling GITHUB_ACTION vs. GITHUB_ACTIONS). Seems it's fixed and we are going to have token-less coverage uploads working: -> https://github.com/codecov/codecov-action/issues/102
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on pull request #10143: Bring back coverage
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #10143:
URL: https://github.com/apache/airflow/pull/10143#issuecomment-669020037
This in is green as well. We need to merge to master to be able to get notified
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org