-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
Fix API docs inconsistencies #3
Conversation
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.
Looks great! We can replace appendTo
with append
on lines 137 & 138 and it is good to be merged 👌
assignWith(10)('a')({ a: 1, b: 2 }) //=> { a: 10, b: 2} | ||
assign(10, 'a', { a: 1, b: 2 }) //=> { a: 10, b: 2} | ||
assign(10, 'a')({ a: 1, b: 2 }) //=> { a: 10, b: 2} | ||
assign(10)('a')({ a: 1, b: 2 }) //=> { a: 10, b: 2} |
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.
👌
@@ -315,7 +313,7 @@ Accepts a number `n`, a function `fn`, and returns a wrapper function of `fn` wi | |||
```js | |||
const array2 = binary(array) | |||
|
|||
array2(1, 2) //=> 1 2 | |||
array2(1, 2) //=> [1, 2] |
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.
👌
drop(3, 'foo..') //=> '...' | ||
drop(3)('foo..') //=> '...' | ||
drop(3, 'foo...') //=> '...' | ||
drop(3)('foo...') //=> '...' |
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.
👌
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.
Thank you a lot for taking the time to contribute : )
No problem, I'll check the rest of the docs later. Do you need any help writing tests or anything else? |
I was reading through the docs and noticed some inconsistencies:
appendTo
calls inappend
examplesassignWith
calls instead ofassign
inassign
examplesarray2(1, 2)
drop3
examples weren't correctI haven't finished reading the docs (I'm at work and it's a lot to read through) but I can check the rest for inconsistencies later if it could be of help.