[dolphinscheduler-sdk-python] branch main updated: [ci] Avoid schedule event upload too many files to codecov (#29)

[zh...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

zhongjiajie pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/dolphinscheduler-sdk-python.git


The following commit(s) were added to refs/heads/main by this push:
     new 79afa8b  [ci] Avoid schedule event upload too many files to codecov (#29)
79afa8b is described below

commit 79afa8badbbf6dc26a78ec41fff7e6292400a871
Author: Jay Chung <zh...@gmail.com>
AuthorDate: Fri Nov 25 13:50:57 2022 +0800

    [ci] Avoid schedule event upload too many files to codecov (#29)
    
    codecov have 100 limit in each commit, and then if will failed the
    upload process, so we should not upload during schedule, because
    6 day not change code base will reach the limit. see more detail
    in https://community.codecov.com/t/ci-failure-due-to-too-many-uploads-to-this-commit/2587/7
---
 .github/workflows/ci.yaml | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
index 684cd3d..c640cad 100644
--- a/.github/workflows/ci.yaml
+++ b/.github/workflows/ci.yaml
@@ -97,6 +97,11 @@ jobs:
         run: |
           python -m tox -vv -e code-test
       - uses: codecov/codecov-action@v3
+        # Codecov have a 100-upload limit per commit, and there are 3 * 6 files upload each time run pytest,
+        # We should not run upload in schedule GitHub event, because the sixth day we do not change our code
+        # and the upload limit will be reached 3 * 6 * 6. For more detail can see:
+        # https://community.codecov.com/t/ci-failure-due-to-too-many-uploads-to-this-commit/2587/7
+        if: ${{ github.event_name != 'schedule' }}
         with:
           token: ${{ env.CODECOV_TOKEN }}
           files: ./coverage.xml