-
Notifications
You must be signed in to change notification settings - Fork 186
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
Ignore Default condition should be last one
#421
Comments
Can you provide an example, I want to investigate, thank you |
I sure can! I have tossed together a quick example :) |
@scagood I see, let's change the logic and will be align with Node.js, just remove it our |
🎉 Thank you @alexander-akait! |
I am looking for a way to more closely mimic the nodejs resolution as in both eslint-community/eslint-plugin-n#182 and eslint-community/eslint-plugin-n#242 we have packages that don't correctly implement the exports order. However, they do resolve correctly when called in either
import
orrequire
.When debugging this initially I discoved the error was throw here: https://github.com/webpack/enhanced-resolve/blob/main/lib/util/entrypoints.js#L474-L476
What I am hoping for is that we can (via a passed in option) disable the throw?
I am very willing to submit code if that would help!
The text was updated successfully, but these errors were encountered: