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 SecretKey::public_key() shortcut #52

Merged
merged 1 commit into from
Jun 14, 2021

Conversation

fjarri
Copy link
Collaborator

@fjarri fjarri commented Jun 13, 2021

Paired with nucypher/pyUmbral#271

See the rationale there. Briefly, this is a more used way, and it coincides with RustCrypto API

@codecov-commenter
Copy link

codecov-commenter commented Jun 13, 2021

Codecov Report

Merging #52 (78e55c5) into master (c9c5770) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #52   +/-   ##
=======================================
  Coverage   83.77%   83.77%           
=======================================
  Files          11       11           
  Lines         912      912           
=======================================
  Hits          764      764           
  Misses        148      148           
Impacted Files Coverage Δ
umbral-pre/src/capsule.rs 88.98% <100.00%> (ø)
umbral-pre/src/capsule_frag.rs 86.02% <100.00%> (ø)
umbral-pre/src/key_frag.rs 81.05% <100.00%> (ø)
umbral-pre/src/keys.rs 74.16% <100.00%> (ø)
umbral-pre/src/pre.rs 93.33% <100.00%> (ø)

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 c9c5770...78e55c5. Read the comment docs.

@fjarri fjarri added this to the v0.2.0 milestone Jun 13, 2021
@fjarri fjarri merged commit 16ce248 into nucypher:master Jun 14, 2021
@fjarri fjarri deleted the public-key-shortcut branch June 14, 2021 20:55
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.

None yet

3 participants