-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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 docs to path
.
#3010
Add docs to path
.
#3010
Conversation
Such suggestions are always welcome! In this case, I disagree on the details. I'll try to think of some more comprehensive text this weekend. |
👍 Let me know if there's more I can do to help. |
Yeah, sorry. I dropped this entirely. I will look again. |
source/path.js
Outdated
@@ -11,7 +13,7 @@ import paths from './paths'; | |||
* @typedefn Idx = String | Int | |||
* @sig [Idx] -> {a} -> a | Undefined | |||
* @param {Array} path The path to use. | |||
* @param {Object} obj The object to retrieve the nested property from. | |||
* @param {Object} obj The object or array to retrieve the nested property from. | |||
* @return {*} The data at `path`. | |||
* @see R.prop, R.nth |
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.
We should also mention assocPath
/dissocPath
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.
Done.
fdda00c
to
93e14f0
Compare
@CrossEye I rebased against the master branch and added your suggestions. |
Oops I totally forgot about this, thanks for carrying it across the finish line, @customcommander! |
Addresses discussion and suggestions in #3007 to clarify documentation of
path
.I'm not sure if any of these doc edits would be welcome, but I thought I'd at least offer them. Feel free to tweak or reject if it doesn't match the style.