Skip to content
This repository was archived by the owner on Feb 22, 2019. It is now read-only.

deserialize the key #111

Merged
merged 3 commits into from
Oct 8, 2013
Merged

deserialize the key #111

merged 3 commits into from
Oct 8, 2013

Conversation

wwwy3y3
Copy link
Contributor

@wwwy3y3 wwwy3y3 commented Jul 10, 2013

if I select the key in my query, the key should also be deserialized

@devdazed
Copy link
Contributor

can we write a test for this and ensure that the existing tests still pass.

@wwwy3y3
Copy link
Contributor Author

wwwy3y3 commented Jul 10, 2013

ok I'll do that soon

@wwwy3y3
Copy link
Contributor Author

wwwy3y3 commented Jul 16, 2013

Hi, I write a uuid key deserializing test, you may take a look
also I fix a bug that if I set cqlVersion to '2.0.0', it will still execute cql3

@wwwy3y3
Copy link
Contributor Author

wwwy3y3 commented Jul 23, 2013

@devdazed if there is anything wrong with code, just tell me, and I'll fix it
otherwise, I think it's good to merge
also, I think some pull requests should also be merged and update the version on NPM
Helenus is an important module in my project
thx!

devdazed added a commit that referenced this pull request Oct 8, 2013
@devdazed devdazed merged commit 1c9203f into simplereach:master Oct 8, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants