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

Fix KeyError caused by wrong key type. #450 #452

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fabiocaccamo
Copy link
Contributor

@fabiocaccamo fabiocaccamo commented Dec 7, 2022

Fix KeyError caused by wrong key type. #450

@fabiocaccamo fabiocaccamo changed the title Fix KeyError. Fix KeyError. #450 Dec 7, 2022
@fabiocaccamo fabiocaccamo changed the title Fix KeyError. #450 Fix KeyError (issue #450). Dec 7, 2022
@fabiocaccamo fabiocaccamo marked this pull request as ready for review December 7, 2022 09:43
@fabiocaccamo fabiocaccamo changed the title Fix KeyError (issue #450). Fix KeyError. #450 Dec 7, 2022
@fabiocaccamo fabiocaccamo changed the title Fix KeyError. #450 Fix KeyError caused by wrong key type. #450 Dec 7, 2022
@jogo
Copy link
Contributor

jogo commented Dec 7, 2022

thanks for the fix, wouldn't this just surprises the issue not fix it?

mind adding a unit test and your name to the credits https://github.com/pinterest/pymemcache#credits

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.

None yet

2 participants