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

cleanup: use PublicKey, SecretKey, Signature from wrapped crypto module #93

Merged
merged 1 commit into from
May 17, 2020

Conversation

okdistribute
Copy link
Contributor

Checklist

  • tests pass
  • tests and/or benchmarks are included
  • documentation is changed or added

Context

Instead of using the ed25519_dalek directly in feed.rs, import it from the crypto wrapper. See #6

This is still forwarding the type though; to fully complete the task this issue calls for wrapping the type as well.

Semver Changes

Patch

@bltavares bltavares merged commit 6f2dbaf into datrs:master May 17, 2020
@bltavares
Copy link
Member

Thank you! Merged and released on the beta versions

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.

2 participants