Add "bun" export condition to common export conditions#48407
Add "bun" export condition to common export conditions#48407jcbhmr wants to merge 1 commit intonodejs:mainfrom jcbhmr:patch-1
Conversation
|
Review requested:
|
|
Rather than expanding this section to every platform, it may be better at this point to reference the list at https://un5mfhvmgtmm6fxwp7ubex3tb6490w1xyjjuwnmq7axdg.julianrbryant.com/ directly here. |
|
Good point! Though I note that https://un5mfhvmgtmm6fxwp7ubex3tb6490w1xyjjuwnmq7axdg.julianrbryant.com/ doesn't include the "types" condition which the Node.js docs do. It also doesn't include "browser", "development" or "production". Maybe it would be best to keep only Node.js-ish stuff like this (below)?
note the lack of "deno" and "bun" |
|
That sounds sensible to me - now that these keys have a definition elsewhere, it would make sense to avoid duplication for them. |
I need to change my commit message because I read https://github.com/nodejs/node/blob/HEAD/doc/contributing/pull-requests.md#commit-message-guidelines after making the tiny change 🤣
For reference on the "bun" export condition: https://un5h3hugw2vg.julianrbryant.com/docs/runtime/modules#resolution
