Skip to content

Conversation

DaniPopes
Copy link
Member

To avoid breaking changes in the future wrap DefaultHashBuilder and DefaultHasher in opaque new-type structs that only implement the required traits. If you need seeded initializer or anything else you should change the S hash builder.

@github-project-automation github-project-automation bot moved this to Reviewed in Alloy Aug 31, 2025
@DaniPopes DaniPopes force-pushed the dani/opaque-hashbuilder branch from 3a74de1 to fdafa6e Compare August 31, 2025 14:00
@DaniPopes DaniPopes enabled auto-merge (squash) August 31, 2025 14:00
@DaniPopes DaniPopes force-pushed the dani/opaque-hashbuilder branch from a15483a to 43d989a Compare September 2, 2025 13:58
@DaniPopes DaniPopes merged commit 401c12f into main Sep 2, 2025
31 checks passed
@DaniPopes DaniPopes deleted the dani/opaque-hashbuilder branch September 2, 2025 14:04
@github-project-automation github-project-automation bot moved this from Reviewed to Done in Alloy Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants