Skip to content

ecdsa: use blanket impl from signature 3 #945

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

Merged
merged 4 commits into from
Apr 23, 2025

Conversation

baloo
Copy link
Member

@baloo baloo commented Apr 21, 2025

This is stacked on #944

This makes use of RustCrypto/traits#1827

@baloo baloo marked this pull request as draft April 21, 2025 21:38
@baloo baloo force-pushed the baloo/signature/blanket-impl branch 2 times, most recently from 326d663 to fa55b79 Compare April 21, 2025 22:04
@baloo baloo force-pushed the baloo/signature/blanket-impl branch 2 times, most recently from 7b989c1 to 1a6f687 Compare April 22, 2025 21:48
@baloo baloo force-pushed the baloo/signature/blanket-impl branch from 1a6f687 to f9c773e Compare April 22, 2025 22:17
@baloo baloo marked this pull request as ready for review April 22, 2025 22:19
Comment on lines 9 to 10
### Changed
- `DigestPrimitive` was removed from hazmat ([#945])
Copy link
Member

Choose a reason for hiding this comment

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

Perhaps put the removal under ### Removed

Copy link
Member Author

Choose a reason for hiding this comment

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

Well, it was not removed per se. I changed the wording of the line instead.

@baloo baloo force-pushed the baloo/signature/blanket-impl branch from 0d49221 to 774419c Compare April 23, 2025 16:37
@tarcieri tarcieri merged commit 8e6bb26 into RustCrypto:master Apr 23, 2025
75 checks passed
@baloo baloo deleted the baloo/signature/blanket-impl branch April 23, 2025 17:17
baloo added a commit to baloo/signatures that referenced this pull request May 4, 2025
tarcieri pushed a commit that referenced this pull request May 4, 2025
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