Skip to content

Fix: Names the Experimental ESM support warning#7069

Open
NormalGaussian wants to merge 1 commit intoyarnpkg:masterfrom
NormalGaussian:fix/name-esm-warning
Open

Fix: Names the Experimental ESM support warning#7069
NormalGaussian wants to merge 1 commit intoyarnpkg:masterfrom
NormalGaussian:fix/name-esm-warning

Conversation

@NormalGaussian
Copy link
Copy Markdown
Contributor

@NormalGaussian NormalGaussian commented Mar 11, 2026

Names the ESM experimental loading warning

What's the problem this PR addresses?

Simple justification: unnamed logs should be avoided

More complex justifiation:
Lots of workspaces now are esm by default. This warning will appear on all of them. While it is "true", there is a strong desire to filter it out so that it does not:

Unnamed logs cannot be filtered.

How did you fix it?

I've added a new name for the warning, and made the warning use that name

Checklist

  • I have read the Contributing Guide.
  • I have set the packages that need to be released for my changes to be effective.
  • I will check that all automated PR checks pass before the PR gets reviewed.

@NormalGaussian
Copy link
Copy Markdown
Contributor Author

It may be desirable to use an EXPERIMENTAL name rather than one so specific?

@clemyan
Copy link
Copy Markdown
Member

clemyan commented Mar 16, 2026

Yeah, probably EXPERIMENTAL to cover all experimental features.

@NormalGaussian
Copy link
Copy Markdown
Contributor Author

I've updated it to be EXPERIMENTAL

@clemyan
Copy link
Copy Markdown
Member

clemyan commented Mar 17, 2026

Please also add a simple description to packages/docusaurus/docs/advanced/01-general-reference/error-codes.mdx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants