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

add codecov config #3037

Merged
merged 3 commits into from
Oct 13, 2021
Merged

add codecov config #3037

merged 3 commits into from
Oct 13, 2021

Conversation

HarrisChu
Copy link
Contributor

refer to https://docs.codecov.com/docs/comparing-commits

default allow_coverage_offsets is false.

@HarrisChu HarrisChu added the ready-for-testing PR: ready for the CI test label Oct 12, 2021
jievince
jievince previously approved these changes Oct 12, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #3037 (17c9114) into master (e642c05) will decrease coverage by 0.03%.
The diff coverage is 36.84%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3037      +/-   ##
==========================================
- Coverage   84.34%   84.31%   -0.04%     
==========================================
  Files        1283     1283              
  Lines      113684   113693       +9     
==========================================
- Hits        95891    95857      -34     
- Misses      17793    17836      +43     
Impacted Files Coverage Δ
src/graph/visitor/DeduceTypeVisitor.h 100.00% <ø> (ø)
src/storage/admin/AdminTaskManager.h 100.00% <ø> (ø)
src/storage/admin/AdminTaskProcessor.cpp 0.00% <0.00%> (ø)
src/storage/admin/AdminTaskManager.cpp 49.55% <34.28%> (-1.18%) ⬇️
src/graph/validator/AdminJobValidator.cpp 93.75% <100.00%> (ø)
src/storage/index/LookupBaseProcessor-inl.h 89.22% <100.00%> (ø)
src/kvstore/raftex/Host.cpp 63.37% <0.00%> (-11.63%) ⬇️
src/common/thrift/ThriftClientManager-inl.h 82.97% <0.00%> (-6.39%) ⬇️
...eta/processors/session/SessionManagerProcessor.cpp 74.52% <0.00%> (-4.46%) ⬇️
src/codec/RowReaderWrapper.h 94.23% <0.00%> (-1.93%) ⬇️
... and 21 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 64721f1...17c9114. Read the comment docs.

yixinglu
yixinglu previously approved these changes Oct 12, 2021
codecov.yml Show resolved Hide resolved
@HarrisChu HarrisChu dismissed stale reviews from yixinglu and jievince via d46c420 October 12, 2021 04:04
@jievince jievince requested a review from yixinglu October 12, 2021 06:11
@Sophie-Xie Sophie-Xie added this to the v2.7.0 milestone Oct 12, 2021
@yixinglu yixinglu merged commit 258e589 into vesoft-inc:master Oct 13, 2021
Sophie-Xie pushed a commit that referenced this pull request Oct 13, 2021
* add codecov config

* add copyright

Co-authored-by: Yee <2520865+yixinglu@users.noreply.github.com>
@Sophie-Xie Sophie-Xie added the cherry-pick-v2.6 PR: need cherry-pick to this version label Oct 13, 2021
critical27 pushed a commit that referenced this pull request Oct 13, 2021
@Sophie-Xie Sophie-Xie modified the milestones: v2.7.0, v2.6.0 Oct 15, 2021
@HarrisChu HarrisChu deleted the codecov_yml branch November 30, 2021 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick-v2.6 PR: need cherry-pick to this version ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants