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

zcash_keys: Remove HdSeedFingerprint as it duplicates zip32::fingerprint::SeedFingerprint' #1274

Merged
merged 1 commit into from
Mar 14, 2024

Conversation

nuttycom
Copy link
Contributor

No description provided.

@nuttycom nuttycom force-pushed the use_zip32_seed_fingerprint branch from 7197737 to bec66f0 Compare March 14, 2024 23:01
@str4d str4d added this to the Librustzcash Zashi 1.0 milestone Mar 14, 2024
str4d
str4d previously approved these changes Mar 14, 2024
Copy link
Contributor

@str4d str4d left a comment

Choose a reason for hiding this comment

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

utACK bec66f0

Copy link
Contributor

@str4d str4d left a comment

Choose a reason for hiding this comment

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

Re-utACK cc990b6

Copy link

codecov bot commented Mar 14, 2024

Codecov Report

Attention: Patch coverage is 68.75000% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 63.45%. Comparing base (be312f8) to head (cc990b6).

Files Patch % Lines
zcash_client_sqlite/src/lib.rs 55.55% 4 Missing ⚠️
zcash_client_sqlite/src/wallet.rs 75.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1274      +/-   ##
==========================================
- Coverage   63.50%   63.45%   -0.05%     
==========================================
  Files         121      121              
  Lines       13711    13702       -9     
==========================================
- Hits         8707     8695      -12     
- Misses       5004     5007       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@nuttycom nuttycom merged commit c3d82b2 into zcash:main Mar 14, 2024
21 of 25 checks passed
@nuttycom nuttycom deleted the use_zip32_seed_fingerprint branch March 14, 2024 23:37
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