-
Notifications
You must be signed in to change notification settings - Fork 119
Support for Tuples? #22
Comments
Patches accepted. |
FWIW, what makes this hard is that you'd need a deserializer for |
I think it's pretty awesome that this is issue #22. |
Also, what would they look like? { "_1": 1, "_2": "two", "_3": 3.0 } ? |
Oh, hell no. [1, "two", 3.0] Coda Hale On Thursday, September 15, 2011 at 1:49 PM, Alex Cruise wrote:
|
This should be open now, given the revert, correct? I'm very interested in getting this to work for Jerkson. Can you clarify how d592703 with 2.9.1 enables this scenario? I tried applying the commit, but it didn't seem to fix the problem for tuples. |
That commit was indeed unrelated. |
I've attempted an implementation of this as josephw/jerkson@c189f6c and opened a pull request. |
Right now, Tuples are not supported by Jerkson, though it seems like they could easily be added. Your CaseClassSerializer/Deserializer classes are remarkably close to what is needed.
The text was updated successfully, but these errors were encountered: