potiuk opened a new pull request #10208: URL: https://github.com/apache/airflow/pull/10208
With this change we only upload coverage report in the case when all tests were successuful and actually executed. This means that coverage report will not be run when the job gets canceled. Currently a lof of coverage reports gathered contain far less coverage because when static check fails or docs some test jobs could already submit their coverage - resulting in partial coverage reports. With this change we also remove comment from coverage report and replace it with (for now) informational status message published to github. If we see that it works, we can change it to a PR-failing status if coverage drops for a given PR. This way we might get our coverage monotonously increasing :). --- **^ 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: [email protected]
