-
-
Notifications
You must be signed in to change notification settings - Fork 106
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
FIX TypeError: unhashable type #319
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It needs to have a unit test and run pre-commit for stylistic fixes. Also add changelog entry.
I am just copying things together so some help would be appreciated.
See my question above. |
Tests look good to me. |
Thanks @spacemanspiff2007! |
Do you have any idea when the next version will be released? |
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
removed Optional Co-authored-by: Hood Chatham <roberthoodchatham@gmail.com>
1a51126
to
8ad98ef
Compare
Probably in the next day or two. |
Attempt to fix #318.
The access to
_TYPES_DICT
should be wrapped so the error for unhashable type is captured.This is just a quick hack that fixes the issue for me and there sure is a more elegant way of doing it ...