-
Notifications
You must be signed in to change notification settings - Fork 2
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
immutable.js #2
Comments
Thanks @gilbox I'm definitely going to do this very soon. In fact I'm going to release another library importing which will make Immutable.js data structures transducers friendly. I was going to do it sooner but end up changing some internals of this library to become compatible with somewhat standardised transducers protocol which will affect this: |
I did not have a chance to document this in readme yet but now there are tests that illustrate how support for Immutable.js datas structures can be implemented: https://github.com/Gozala/transducers/blob/master/src/test/index.js#L519-L599 |
awesome! thanks so much. |
@gilbox btw I just published this version to npm (version before had a different API) |
@Gozala do you still have plan to create library for immutable.js? Maybe you already have some draft of it that you can publish, so other people can contribute. |
@ArtemKozlov I do but I have not had chance to do it yet got busy with https://github.com/Gozala/typed-immutable and figuring out how to integrate it into an immutable itself. I'll try to get it going sometime next week or maybe even this weekend! In terms of draft it's basically whatever you can see in the tests linked, it's just matter of defining those for all the data types though. |
This looks really cool. Could you add an example to the README illustrating how to use it with immutable-js?
The text was updated successfully, but these errors were encountered: