Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@alwayys-afk
Copy link
Contributor

@alwayys-afk alwayys-afk commented Feb 19, 2025

Fixes #1318

@dsgallups insightfully pointed out that the ToSchema implementation for HashSets and HashMaps does not support custom hash functions. This PR augments the code to allow for custom hash functions, providing greater flexibility in schema generation.

@dsgallups
Copy link

How thoughtful! Thanks @alwayys-afk!

Copy link

@dsgallups dsgallups left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Owner

@juhaku juhaku left a comment

Choose a reason for hiding this comment

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

💯

@juhaku juhaku merged commit 9b17a2e into juhaku:master May 26, 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.

HashSet without RandomState does not implement ToSchema

3 participants