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: upload codecov reports all at once #3856

Merged
merged 8 commits into from
Jul 1, 2020

Conversation

rainydio
Copy link
Contributor

@rainydio rainydio commented Jun 30, 2020

Summary

New job that depends on all other jobs in workflow and uploads all codecov reports at once.

Checklist

  • Ready to be merged

@ghost ghost added Complexity: High labels Jun 30, 2020
@codecov
Copy link

codecov bot commented Jun 30, 2020

Codecov Report

Merging #3856 into develop will increase coverage by 5.08%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #3856      +/-   ##
===========================================
+ Coverage    87.06%   92.14%   +5.08%     
===========================================
  Files          620      621       +1     
  Lines        14816    14826      +10     
  Branches      1852     1831      -21     
===========================================
+ Hits         12899    13662     +763     
+ Misses        1706      952     -754     
- Partials       211      212       +1     
Flag Coverage Δ
#functional 6.07% <ø> (+0.06%) ⬆️
#integration 9.57% <ø> (-0.01%) ⬇️
#unit 90.21% <ø> (+5.08%) ⬆️
Impacted Files Coverage Δ
packages/core-p2p/src/socket-server/server.ts 100.00% <0.00%> (ø)
...-p2p/src/socket-server/plugins/whitelist-forger.ts 85.71% <0.00%> (ø)
packages/core-p2p/src/peer-processor.ts 91.11% <0.00%> (+0.20%) ⬆️
...e-test-framework/src/matchers/functional/entity.ts 93.54% <0.00%> (+3.22%) ⬆️
packages/core-snapshots/src/events.ts 100.00% <0.00%> (+100.00%) ⬆️
packages/core-snapshots/src/verifier.ts 100.00% <0.00%> (+100.00%) ⬆️
packages/core-snapshots/src/workers/worker.ts 100.00% <0.00%> (+100.00%) ⬆️
packages/core-snapshots/src/ioc/identifiers.ts 100.00% <0.00%> (+100.00%) ⬆️
packages/core-snapshots/src/database-service.ts 100.00% <0.00%> (+100.00%) ⬆️
packages/core-snapshots/src/service-provider.ts 100.00% <0.00%> (+100.00%) ⬆️
... and 20 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 1a81aee...5ee2221. Read the comment docs.

@rainydio rainydio changed the title chore: upload all codecov reports at once chore: upload codecov reports all at once Jun 30, 2020
@rainydio rainydio force-pushed the fix/github-workflow/codecov branch from cbc7cc1 to 78b1511 Compare June 30, 2020 22:01
@rainydio rainydio force-pushed the fix/github-workflow/codecov branch from f4f2699 to 5ee2221 Compare June 30, 2020 22:37
@rainydio rainydio marked this pull request as ready for review June 30, 2020 23:07
@faustbrian faustbrian merged commit 60624a3 into develop Jul 1, 2020
@ghost ghost deleted the fix/github-workflow/codecov branch July 1, 2020 02:31
@ghost ghost removed the Status: Needs Review label Jul 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants