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

chore: support codecov #186

Merged
merged 3 commits into from
Aug 12, 2022
Merged

chore: support codecov #186

merged 3 commits into from
Aug 12, 2022

Conversation

GuangmingLuo
Copy link
Member

@GuangmingLuo GuangmingLuo commented Aug 9, 2022

What type of PR is this?
chore: support codecov and file ignore

What this PR does / why we need it (en: English/zh: Chinese):
en: support codecov
zh: 支持 codecov

Duslia
Duslia previously approved these changes Aug 10, 2022
@codecov
Copy link

codecov bot commented Aug 10, 2022

Codecov Report

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

@@            Coverage Diff             @@
##             develop     #186   +/-   ##
==========================================
  Coverage           ?   58.98%           
==========================================
  Files              ?       79           
  Lines              ?     8000           
  Branches           ?        0           
==========================================
  Hits               ?     4719           
  Misses             ?     2935           
  Partials           ?      346           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@welkeyever
Copy link
Member

这个ut是不是应该豁免一些文件夹(example、cmd/hz 这些)?

@GuangmingLuo
Copy link
Member Author

这个ut是不是应该豁免一些文件夹(example、cmd/hz 这些)?

是的,可以在那个 ignore 文件里面补充

Change-Id: I587c29640f38706e09b3fc112d0a89d4c471dc47
@GuangmingLuo GuangmingLuo merged commit d0c1f66 into develop Aug 12, 2022
@GuangmingLuo GuangmingLuo deleted the codecov branch August 12, 2022 02:37
FGYFFFF pushed a commit to FGYFFFF/hertz that referenced this pull request Aug 24, 2022
* add codecov ignore

* add codecov ci config

* add dir ignore (cloudwego#187)

Change-Id: I587c29640f38706e09b3fc112d0a89d4c471dc47

Co-authored-by: Wenju Gao <gaowenju@bytedance.com>
FGYFFFF pushed a commit to FGYFFFF/hertz that referenced this pull request Aug 24, 2022
* add codecov ignore

* add codecov ci config

* add dir ignore (cloudwego#187)

Change-Id: I587c29640f38706e09b3fc112d0a89d4c471dc47

Co-authored-by: Wenju Gao <gaowenju@bytedance.com>
FGYFFFF pushed a commit to FGYFFFF/hertz that referenced this pull request Aug 24, 2022
* add codecov ignore

* add codecov ci config

* add dir ignore (cloudwego#187)

Change-Id: I587c29640f38706e09b3fc112d0a89d4c471dc47

Co-authored-by: Wenju Gao <gaowenju@bytedance.com>
FGYFFFF pushed a commit that referenced this pull request Aug 25, 2022
* add codecov ignore

* add codecov ci config

* add dir ignore (#187)

Change-Id: I587c29640f38706e09b3fc112d0a89d4c471dc47

Co-authored-by: Wenju Gao <gaowenju@bytedance.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants