Update dependencies and clean up the code #86
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR cleans up the
schnorrkel
code a little to prepare for a new release.curve25519-dalek
fromcurve25519-dalek-ng
(it turns out usingcurve25519-dalek-ng
was a bad decision as that fork is effectively dead now)clippy
lintsmusig
as deprecated (as requested by @burdges)This should not yet be released on crates.io. Before we cut a new release I want to do two more things:
curve25519-dalek
and a release is cut, and then bumpschnorrkel
to use that. This should provide massive speedups in the range of ~30%-~50%.substrate
burn-in to make sure nothing's accidentally broken. I might also do some fuzz testing to make sure the behavior's equivalent to the 0.9.1 version currently used bysubstrate
.