Skip to content
This repository has been archived by the owner on Apr 8, 2020. It is now read-only.

Adds id() method to Secp256k1PrivateKey #13

Merged
merged 1 commit into from
Feb 20, 2019

Conversation

AlbertoElias
Copy link
Contributor

Feature parity with ed25519 and rsa

@AlbertoElias
Copy link
Contributor Author

This is blocking libp2p/js-libp2p-keychain#30

@AlbertoElias
Copy link
Contributor Author

Hey @dignifiedquire could you review please?

Copy link
Member

@dignifiedquire dignifiedquire left a comment

Choose a reason for hiding this comment

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

lgtm

@AlbertoElias
Copy link
Contributor Author

@dignifiedquire great! I don't have write permissions. Could you merge and release? Thank you!

Copy link
Member

@vasco-santos vasco-santos left a comment

Choose a reason for hiding this comment

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

LGTM

@vasco-santos
Copy link
Member

@dignifiedquire can we get this merged and released?

@dignifiedquire dignifiedquire merged commit f4dbd62 into libp2p:master Feb 20, 2019
@dignifiedquire
Copy link
Member

published, as 0.3.0 (sorry meant to do it as patch, but feature made me think of minor release)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants