Skip to content
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

errors: include CLI solution in ESM error message #51963

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

DanKaplanSES
Copy link
Contributor

@DanKaplanSES DanKaplanSES commented Mar 4, 2024

  • This error is printed when you run node file.js on a file that contains a static import. It provides two solutions. I've added a third fourth.

- This error is printed when you run `node file.js` on a file that
  contains a static import. It provides two solutions. I've added a
third.
@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/loaders

Sorry, something went wrong.

@nodejs-github-bot nodejs-github-bot added esm Issues and PRs related to the ECMAScript Modules implementation. needs-ci PRs that need a full CI run. labels Mar 4, 2024
@aduh95
Copy link
Contributor

aduh95 commented Mar 4, 2024

We could also mention --experimental-detect-module

@DanKaplanSES
Copy link
Contributor Author

We could also mention --experimental-detect-module

Done

@GeoffreyBooth
Copy link
Member

We could also mention --experimental-detect-module

I’m not sure we should be mentioning experimental features here? They’re subject to change or disappearance, after all. In particular I feel like there’s not much reason for us to keep --experimental-default-type now that we have --experimental-detect-module.

@DanKaplanSES
Copy link
Contributor Author

DanKaplanSES commented Mar 5, 2024

I’m not sure we should be mentioning experimental features here? They’re subject to change or disappearance, after all.

That's true. What if this was merged temporarily, and when the experimental features change or disappear, this message can be reverted?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
esm Issues and PRs related to the ECMAScript Modules implementation. needs-ci PRs that need a full CI run.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants