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

cli: add wallet candidate getstate #1986

Merged
merged 2 commits into from
May 31, 2021
Merged

cli: add wallet candidate getstate #1986

merged 2 commits into from
May 31, 2021

Conversation

AnnaShaleva
Copy link
Member

Close #1983.

@AnnaShaleva AnnaShaleva changed the title cli: add wallet candidat getstate cli: add wallet candidate getstate May 28, 2021
@AnnaShaleva AnnaShaleva force-pushed the cli/getaccountstate branch from c8be613 to 1ab491b Compare May 28, 2021 15:09
@codecov
Copy link

codecov bot commented May 28, 2021

Codecov Report

Merging #1986 (1ab491b) into master (9340a70) will increase coverage by 0.08%.
The diff coverage is 59.25%.

❗ Current head 1ab491b differs from pull request most recent head 3ed3a16. Consider uploading reports for the commit 3ed3a16 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1986      +/-   ##
==========================================
+ Coverage   83.18%   83.27%   +0.08%     
==========================================
  Files         285      285              
  Lines       22947    23172     +225     
==========================================
+ Hits        19089    19296     +207     
- Misses       2671     2678       +7     
- Partials     1187     1198      +11     
Impacted Files Coverage Δ
cli/wallet/validator.go 71.73% <59.25%> (-5.19%) ⬇️
pkg/consensus/consensus.go 70.33% <0.00%> (-0.62%) ⬇️
pkg/network/server.go 70.76% <0.00%> (-0.15%) ⬇️
cli/smartcontract/smart_contract.go 89.98% <0.00%> (+2.88%) ⬆️
cli/cmdargs/parser.go 98.42% <0.00%> (+6.11%) ⬆️

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 9340a70...3ed3a16. Read the comment docs.

@AnnaShaleva AnnaShaleva force-pushed the cli/getaccountstate branch 2 times, most recently from 210c8f8 to 3ed3a16 Compare May 31, 2021 08:03
@AnnaShaleva AnnaShaleva requested a review from roman-khimov May 31, 2021 08:04
@AnnaShaleva AnnaShaleva force-pushed the cli/getaccountstate branch from 3ed3a16 to c2b5459 Compare May 31, 2021 08:19
@roman-khimov roman-khimov merged commit 0677838 into master May 31, 2021
@roman-khimov roman-khimov deleted the cli/getaccountstate branch May 31, 2021 08:39
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.

getAccountState from CLI
3 participants