-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
fix: fail with an informative error message on a file with a broken default import #5413
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
fix: fail with an informative error message on a file with a broken default import #5413
Conversation
Fail with an informative error message on a file with a broken default import.
edb0e34
to
e171a5d
Compare
Nice :) cursory review looks good, I'll dive deep into this tomorrow as I'm still a new maintainer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems reasonable for the intended behavior, but I'm not sure the intended behavior is what we want?
Thanks for starting this though, the change is very clean!
…w the require error in case the import error is ERR_INTERNAL_ASSERTION
@martinslota can you resolve merge conflicts? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Fail with an informative error message on a file with a broken default import.
PR Checklist
status: accepting prs
Overview
Fail with an informative error message on a file with a broken default import.