BLAKE3-team / BLAKE3

the official Rust and C implementations of the BLAKE3 cryptographic hash function
Apache License 2.0
5.07k stars 346 forks source link

Add serde support for Hash behind optional feature #339

Closed rminderhoud closed 1 year ago

rminderhoud commented 1 year ago

Added a new cargo feature serde that when enabled will derive serde::Serialize and serde::Deserialize for the blake3::Hash struct.

Fixes https://github.com/BLAKE3-team/BLAKE3/issues/290

oconnor663 commented 1 year ago

Lots of folks have asked for this over the last couple years, so it probably does make sense to go ahead and include it. Thanks for opening the PR!

rminderhoud commented 1 year ago

Lots of folks have asked for this over the last couple years, so it probably does make sense to go ahead and include it. Thanks for opening the PR!

Thanks, I'm glad you are willing to merge it into main!

oconnor663 commented 1 year ago

Could you rebase on the most recent master? There has been some churn in Cargo.toml.

rminderhoud commented 1 year ago

Sure no problem, rebased with latest master branch

oconnor663 commented 1 year ago

Landed, thanks!