Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature: restrict codecov.yml to ignore files #1050

Merged

Conversation

allencloud
Copy link
Collaborator

Signed-off-by: Allen Sun [email protected]

Ⅰ. Describe what this PR did

This PR adds a codecov.yml file to ignore some test files. Since I think we do not need to test file

  - "extra/*"
  - "vendor/*"
  - "apis/types/*"

Ⅱ. Does this pull request fix one issue?

none

Ⅲ. Describe how you did it

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@allencloud allencloud force-pushed the add-codecov-yml branch 5 times, most recently from d97b1c8 to 79261fa Compare April 3, 2018 17:16
@codecov-io
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (master@97fbd00). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##             master   #1050   +/-   ##
========================================
  Coverage          ?   16.1%           
========================================
  Files             ?     139           
  Lines             ?    8493           
  Branches          ?       0           
========================================
  Hits              ?    1368           
  Misses            ?    7022           
  Partials          ?     103

@allencloud
Copy link
Collaborator Author

LGTM

@pouchrobot pouchrobot added the LGTM one maintainer or community participant agrees to merge the pull reuqest. label Apr 3, 2018
@allencloud allencloud merged commit 04943ff into AliyunContainerService:master Apr 3, 2018
@allencloud allencloud deleted the add-codecov-yml branch April 3, 2018 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature LGTM one maintainer or community participant agrees to merge the pull reuqest. size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants