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

feature: allow for custom error messaging in ModuleScopePlugin #11978

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

Conversation

Aghassi
Copy link

@Aghassi Aghassi commented Jan 23, 2022

This is purely related to #11977 as a proof of concept for what I'm proposing. It would allow folks who directly use this webpack plugin to customize the error output. Broader use of this error through other means in CRA would need more development.

This change allows the developer to pass in an error message so that they can provide a custom error message for the missing module in the event that it is ejected from CRA OR when the developer is using react-dev-utils directly for their own use case (which is my current way of using this). So this allows developers who use this to get a more fine grain result for their end users with a better error message.

This is purely related to facebook#11977 as a proof of concept for what I'm proposing. It would allow folks who directly use this webpack plugin to customize the error output. Broader use of this error through other means in CRA would need more development.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants