Skip to content

Conversation

@winder
Copy link
Contributor

@winder winder commented Aug 26, 2021

Summary

This PR comes after #2808

Test Plan

CI

@codecov-commenter
Copy link

codecov-commenter commented Aug 26, 2021

Codecov Report

Merging #2810 (97d5b74) into feature/partkey (0aec085) will increase coverage by 0.05%.
The diff coverage is 40.67%.

Impacted file tree graph

@@                 Coverage Diff                 @@
##           feature/partkey    #2810      +/-   ##
===================================================
+ Coverage            47.45%   47.50%   +0.05%     
===================================================
  Files                  352      352              
  Lines                56868    56903      +35     
===================================================
+ Hits                 26984    27031      +47     
+ Misses               26848    26834      -14     
- Partials              3036     3038       +2     
Impacted Files Coverage Δ
data/accountManager.go 0.00% <0.00%> (ø)
data/account/msgp_gen.go 46.30% <52.00%> (+12.21%) ⬆️
data/account/participationRegistry.go 75.96% <71.42%> (-0.32%) ⬇️
data/account/participation.go 41.00% <75.00%> (+0.79%) ⬆️
node/node.go 26.67% <75.00%> (+0.23%) ⬆️
ledger/acctupdates.go 62.55% <0.00%> (-0.09%) ⬇️
catchup/service.go 69.35% <0.00%> (+0.77%) ⬆️
network/wsPeer.go 75.48% <0.00%> (+0.83%) ⬆️
data/transactions/verify/txn.go 45.08% <0.00%> (+0.89%) ⬆️
cmd/tealdbg/debugger.go 73.86% <0.00%> (+1.00%) ⬆️
... and 4 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 0aec085...97d5b74. Read the comment docs.

@winder winder requested a review from tsachiherman September 13, 2021 19:15
@winder winder marked this pull request as ready for review September 13, 2021 19:15
@winder winder merged commit 504cbc8 into algorand:feature/partkey Sep 22, 2021
@winder winder deleted the will/participationregistry-installdelete branch September 22, 2021 19:45
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.

3 participants