Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Enhancement] Add support for MapSets #147

Open
kerryb opened this issue Aug 27, 2024 · 1 comment
Open

[Enhancement] Add support for MapSets #147

kerryb opened this issue Aug 27, 2024 · 1 comment

Comments

@kerryb
Copy link

kerryb commented Aug 27, 2024

It looks like Hammox doesn’t currently support typespecs that use MapSet.t, because these use Elixir’s :sets module internally, and that declares its types as records.

I’ve created a trivial project to demonstrate the issue – I’m not sure I’ve got the skills to figure out how to fix it, but I might give it a try if I get time.

@brettbeatty
Copy link

Just ran into this same problem. Looks like the underlying type for MapSet now is :sets.set/1, which is a union of both old version (record) and new (map). So even though MapSet will always have the map version the typespec includes a record version.

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

No branches or pull requests

2 participants