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 load localWitnessAccountAddress for keystore mode #2457

merged 1 commit into from Sep 9, 2019


Copy link

@sean-liu55 sean-liu55 commented Sep 9, 2019

What does this PR do?
add load localWitnessAccountAddress for keystore mode

Why are these changes required?

This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

@sean-liu55 sean-liu55 requested a review from zergweak Sep 9, 2019
Copy link

@codecov-io codecov-io commented Sep 9, 2019

Codecov Report

Merging #2457 into develop_v3.6.5 will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@                 Coverage Diff                  @@
##             develop_v3.6.5    #2457      +/-   ##
- Coverage             42.44%   42.43%   -0.01%     
  Complexity             4383     4383              
  Files                   538      538              
  Lines                 32818    32825       +7     
  Branches               3633     3635       +2     
  Hits                  13928    13928              
- Misses                17534    17541       +7     
  Partials               1356     1356
Impacted Files Coverage Δ Complexity Δ
src/main/java/org/tron/core/config/args/ 61% <0%> (-0.55%) 146 <0> (ø)

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 e3f8abc...d173284. Read the comment docs.

@renchenchang renchenchang merged commit 0b4b01d into develop_v3.6.5 Sep 9, 2019
7 checks passed
@shydesky shydesky deleted the HotFix/witness_keystore2 branch Nov 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants