docs: document package.json's "exports" field
#7323
Conversation
| "./*": "./*.js", | ||
| "./*.js": "./*.js", | ||
| "./foo": "./path/to/foo.js" |
There was a problem hiding this comment.
ideally an example would include "./package.json", so tools can access the package.json file, and would also show multiple examples including conditions, arrays, etc.
There was a problem hiding this comment.
thank you for the feedback, I've now documented conditional exports.
I have also documented fallbacks although I have not yet added examples for fallbacks.
is the thing about conditional exports satisfactory (like the video game)?
|
sorry about the force push, I started writing in GitHub's editor, copy pasted to vscode to carry on editing there, copy pasted back to GitHub's editor to commit but I apparently failed to paste |
| ### exports | ||
|
|
||
| The exports field is an object that maps entry points to modules. This field is | ||
| supported by Node.js versions including and higher than 12. It acts as a more |
There was a problem hiding this comment.
higher than 12
see https://npmjs.com/node-exports-info - there’s actually a pretty complex set of feature support and node version matrix.
There was a problem hiding this comment.
do you think it would be better to take out the "higher than 12" comment since the version of NPM these docs are for only supports Node.js ^18.17.0 || >=20.5.0 anyway?
There was a problem hiding this comment.
yes - or maybe, "is supported by modern node.js versions"? also this should link to the node documentation of the field, which might be better than having examples here.
There was a problem hiding this comment.
I've added your suggestions, how's this so far?
|
I have made all requested changes. Is this any closer to being merged? |
|
Given how in-depth a true documentation of |
Yup. I agree and have linked to Node.js's official docs on the |
Document
package.json's"exports"field.