Skip to content

ecdsa: use blanket impl from signature 3#945

Merged
tarcieri merged 4 commits intoRustCrypto:masterfrom
baloo:baloo/signature/blanket-impl
Apr 23, 2025
Merged

ecdsa: use blanket impl from signature 3#945
tarcieri merged 4 commits intoRustCrypto:masterfrom
baloo:baloo/signature/blanket-impl

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