Skip to content

Conversation

@csr632
Copy link

@csr632 csr632 commented Aug 19, 2022

The tool used to do named exports detection should be cjs-module-lexer, not esm-module-lexer.
Reference: https://github.com/nodejs/node/blob/5e57d24d325f0aea74394f78ebdc06857cca77b1/doc/api/esm.md

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/modules

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Aug 19, 2022
@richardlau
Copy link
Member

This overlaps with #44262.

@JakobJingleheimer
Copy link
Member

Thanks for the contribution! As Richard mentioned, there is already an approved PR fixing this.

P.S. The links at the bottom of the doc file must be in alphabetical order. The current error is misleading:

Unordered reference ("cjs-module-lexer" should be before "the full specifier path")

The new link you added must indeed be listed above the one immediately above it, but it actually must be far above.

@csr632
Copy link
Author

csr632 commented Aug 20, 2022

Thanks for the review! I am closing this PR.

@csr632 csr632 closed this Aug 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants