Skip to content

Go back to requiring siphash24#297

Merged
inducer merged 1 commit intomainfrom
require-siphash
May 5, 2025
Merged

Go back to requiring siphash24#297
inducer merged 1 commit intomainfrom
require-siphash

Conversation

@inducer
Copy link
Owner

@inducer inducer commented May 5, 2025

@inducer inducer force-pushed the require-siphash branch from 04204cb to fe44d11 Compare May 5, 2025 19:41
@inducer inducer merged commit b2bd225 into main May 5, 2025
17 checks passed
@inducer inducer deleted the require-siphash branch May 5, 2025 20:07
@inducer
Copy link
Owner Author

inducer commented May 5, 2025

@connorjward This having been merged notwithstanding: are there any concerns about this from your end? I'll hold off on rolling a release until I've heard from you.

@connorjward
Copy link

@connorjward This having been merged notwithstanding: are there any concerns about this from your end? I'll hold off on rolling a release until I've heard from you.

We were already using the siphash optional dependency to avoid the warning. I think this will be fine.

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.

siphash24 warning

2 participants