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
add functional auto-curried syntax support #3
Conversation
|
||
```js | ||
const {map, reduce, filter, forEach} = require('objectfn') | ||
|
||
const obj = { foo: 'bar', wow: 'doge' } | ||
|
||
map(obj, (val, key) => val.toUpperCase()) | ||
const upcasedValues = map(obj, (val) => val.toUpperCase()) | ||
console.log(upcasedValues) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why add extra boilerplate here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've developed a habit of making code examples runnable on tonicdev.com but i see there are other ways of doing this so I've removed the boilerplate. I also spotted some errors in the example which I've fixed
This is awesome. You are an es6 magician |
@jescalan fixed some code example errors + removed boilerplate |
Ok cool, looking good. So the question is, do we need a major version for this? Is there any regression at all? |
The addition of the |
supersedes #2