Skip to content
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

time to make each public #17

Closed
buzzdecafe opened this issue Mar 2, 2014 · 2 comments
Closed

time to make each public #17

buzzdecafe opened this issue Mar 2, 2014 · 2 comments

Comments

@buzzdecafe
Copy link
Member

working with the DOM has convinced me of the need for an each method on ramda.
The DOM does not lend itself to immutability. Or referential transparency. Or elegance. so this is one vote for exposing a bit of impurity (gasp) in the lib for the sake of practicality.

Note that since we are not using native array methods (e.g. slice) internally, we can use ramda seamlessly with NodeLists et al.

Thoughts?

@CrossEye
Copy link
Member

CrossEye commented Mar 3, 2014

No major objection, although the public name should probably be forEach. Probably the internal name too. jQuery should not ever be responsible for how we choose to name things!

@buzzdecafe
Copy link
Member Author

done, and .idx too.

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

No branches or pull requests

2 participants