Skip to content
This repository was archived by the owner on Jan 2, 2023. It is now read-only.

Conversation

@taichi
Copy link

@taichi taichi commented Jun 11, 2014

No description provided.

@iconara
Copy link
Owner

iconara commented Jun 11, 2014

Thank you for this PR. If we're going to add #to_msgpack I think it must include support for adding the method to your own classes, like suggested in #13.

The idea right now is to try to merge this gem into msgpack/msgpack-ruby, and that means supporting everything that that gem supports, including this. I'll have to see if I'll merge this PR or implement it as part of the merge.

@taichi
Copy link
Author

taichi commented Jun 11, 2014

this PR cotains very simple case only.
this PR don't contain one argment with calling to_msgpack method.

@iconara
Copy link
Owner

iconara commented Jul 16, 2014

This will happen in msgpack/msgpack-ruby#33

@taichi
Copy link
Author

taichi commented Jul 16, 2014

👍

1 similar comment
@miry
Copy link

miry commented Sep 19, 2014

👍

@niamster
Copy link

Any chance to have it mainline?

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.

4 participants