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

Already on GitHub? Sign in to your account

allow non-ascii keys #6

Merged
merged 2 commits into from Dec 12, 2011

Conversation

Projects
None yet
2 participants
Contributor

jkarneges commented Dec 12, 2011

No description provided.

Contributor

progrium commented Dec 12, 2011

I take it current tests pass? Can you write a test for this change that makes sure non-ascii characters can get through?

Contributor

jkarneges commented Dec 12, 2011

Yep, unicode key test still passes. Added a test for non-ascii now too.

@progrium progrium added a commit that referenced this pull request Dec 12, 2011

@progrium progrium Merge pull request #6 from jkarneges/master
allow non-ascii keys
4502e96

@progrium progrium merged commit 4502e96 into jpadilla:master Dec 12, 2011

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment