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/12/07 09:05:14 UTC

[GitHub] [airflow] potiuk commented on pull request #20087: Upload coverage for PRs to main

potiuk commented on pull request #20087:
URL: https://github.com/apache/airflow/pull/20087#issuecomment-987713100


   This is not a good idea for two reasons:
   
   * Gathering coverage during our tests takes a lot of memory - see #19523. Most of our PRs are run in Public GitHub Runners and gathering coverage will cause OOM errors on public runners.
   
   * We are using selective checks in our PRs. We detect what kind of changes are done in the PR and based on that we run subset of tests. That makes  lcoverage information wrong because those PRs show far less coverage.


-- 
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