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

feat(cli): introduce planet key derive #1268

Merged
merged 11 commits into from
Apr 29, 2021

Conversation

moreal
Copy link
Contributor

@moreal moreal commented Apr 26, 2021

This pull request introdcues planet key convert private-key subcommand to provide conversion private key → public key and private key → address.

@moreal moreal added the tools Related to CLI tools (Libplanet.Tools) label Apr 26, 2021
@moreal moreal marked this pull request as ready for review April 26, 2021 09:56
@codecov
Copy link

codecov bot commented Apr 27, 2021

Codecov Report

Merging #1268 (5d70a0d) into main (b1e3e84) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1268      +/-   ##
==========================================
- Coverage   84.04%   84.02%   -0.03%     
==========================================
  Files         398      398              
  Lines       34110    34110              
==========================================
- Hits        28667    28660       -7     
- Misses       3617     3625       +8     
+ Partials     1826     1825       -1     
Impacted Files Coverage Δ
Libplanet/Net/Swarm.BlockSync.cs 73.89% <0.00%> (-1.58%) ⬇️
Libplanet/Store/DefaultStore.cs 86.02% <0.00%> (-1.45%) ⬇️
Libplanet/Net/Transports/NetMQTransport.cs 78.38% <0.00%> (-0.89%) ⬇️
Libplanet/Net/BlockCompletion.cs 94.20% <0.00%> (-0.87%) ⬇️
Libplanet/Net/Protocols/KademliaProtocol.cs 67.41% <0.00%> (-0.38%) ⬇️
Libplanet.Tests/Net/SwarmTest.Preload.cs 99.29% <0.00%> (+0.28%) ⬆️
Libplanet/Net/Swarm.cs 84.28% <0.00%> (+0.93%) ⬆️
Libplanet/Net/Protocols/KBucket.cs 89.74% <0.00%> (+2.56%) ⬆️

@moreal
Copy link
Contributor Author

moreal commented Apr 27, 2021

Could you review this pull request, @libplanet ?

riemannulus
riemannulus previously approved these changes Apr 27, 2021
@moreal moreal force-pushed the feature/cli/provide-conversion-command branch 2 times, most recently from 9b71469 to 7517a10 Compare April 27, 2021 06:34
@longfin longfin changed the title feat(cli): introduce planet key convert feat(cli): introduce planet key derive Apr 27, 2021
@moreal moreal requested review from dahlia and riemannulus April 28, 2021 04:06
Copy link
Contributor

@dahlia dahlia left a comment

Choose a reason for hiding this comment

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

I hope this is my last review comment.

@moreal moreal force-pushed the feature/cli/provide-conversion-command branch from cdc27e8 to e573ce6 Compare April 28, 2021 07:07
@moreal moreal requested a review from dahlia April 28, 2021 07:09
dahlia
dahlia previously approved these changes Apr 28, 2021
moreal and others added 5 commits April 28, 2021 18:27
Co-authored-by: Hong Minhee (洪 民憙) <[email protected]>
Co-authored-by: Hong Minhee (洪 民憙) <[email protected]>
@moreal moreal force-pushed the feature/cli/provide-conversion-command branch from e573ce6 to ac21b6d Compare April 28, 2021 09:27
@moreal moreal requested a review from dahlia April 28, 2021 22:48
@moreal
Copy link
Contributor Author

moreal commented Apr 29, 2021

Could you review this pull request? @libplanet 🙏🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tools Related to CLI tools (Libplanet.Tools)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants