Skip to content

goal: account deletepartkey subcomand#4788

Merged
winder merged 9 commits into
masterfrom
3796-new-goal-account-deletepartkey-subcommand
Dec 7, 2022
Merged

goal: account deletepartkey subcomand#4788
winder merged 9 commits into
masterfrom
3796-new-goal-account-deletepartkey-subcommand

Conversation

@AlgoStephenAkiki
Copy link
Copy Markdown
Contributor

@AlgoStephenAkiki AlgoStephenAkiki commented Nov 14, 2022

Summary

Resolves #3796

Adds delete part keys helper function

Test Plan

New integration test.

@AlgoStephenAkiki AlgoStephenAkiki self-assigned this Nov 14, 2022
@AlgoStephenAkiki AlgoStephenAkiki changed the title Add delete part keys subcomand New Feature: Add delete part keys subcomand Nov 14, 2022
@AlgoStephenAkiki AlgoStephenAkiki force-pushed the 3796-new-goal-account-deletepartkey-subcommand branch from 73f74aa to 44bf852 Compare November 14, 2022 22:12
@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 14, 2022

Codecov Report

Merging #4788 (007a4d9) into master (ffa688a) will increase coverage by 0.02%.
The diff coverage is 23.07%.

@@            Coverage Diff             @@
##           master    #4788      +/-   ##
==========================================
+ Coverage   53.31%   53.34%   +0.02%     
==========================================
  Files         423      423              
  Lines       53852    53865      +13     
==========================================
+ Hits        28713    28732      +19     
  Misses      22854    22854              
+ Partials     2285     2279       -6     
Impacted Files Coverage Δ
libgoal/libgoal.go 2.59% <0.00%> (-0.02%) ⬇️
cmd/goal/account.go 13.99% <33.33%> (+0.23%) ⬆️
catchup/service.go 69.08% <0.00%> (-0.73%) ⬇️
ledger/acctupdates.go 68.63% <0.00%> (-0.37%) ⬇️
data/transactions/verify/txn.go 73.61% <0.00%> (ø)
ledger/testing/randomAccounts.go 57.23% <0.00%> (+0.61%) ⬆️
ledger/catchpointtracker.go 58.88% <0.00%> (+0.74%) ⬆️
catchup/peerSelector.go 100.00% <0.00%> (+1.04%) ⬆️
ledger/blockqueue.go 87.35% <0.00%> (+1.72%) ⬆️
ledger/tracker.go 78.05% <0.00%> (+3.79%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Eric-Warehime
Eric-Warehime previously approved these changes Nov 15, 2022
Comment thread cmd/goal/account.go Outdated
Comment thread daemon/algod/api/client/restClient.go
Copy link
Copy Markdown
Contributor

@winder winder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In addition to the suggestions, please add a test.

Eric-Warehime
Eric-Warehime previously approved these changes Nov 30, 2022
@AlgoStephenAkiki AlgoStephenAkiki force-pushed the 3796-new-goal-account-deletepartkey-subcommand branch from 3405a13 to 2a5315c Compare December 7, 2022 14:58
@winder winder changed the title New Feature: Add delete part keys subcomand goal: account deletepartkey subcomand Dec 7, 2022
@winder winder merged commit 6e26b0d into master Dec 7, 2022
@winder winder deleted the 3796-new-goal-account-deletepartkey-subcommand branch December 7, 2022 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

New "goal account deletepartkey" subcommand.

3 participants