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

add support for Node.js ESM resolution #2480

Closed
piranna opened this issue Jun 18, 2022 · 3 comments
Closed

add support for Node.js ESM resolution #2480

piranna opened this issue Jun 18, 2022 · 3 comments

Comments

@piranna
Copy link

piranna commented Jun 18, 2022

Similar to eslint-community/eslint-plugin-n#4, add support on import/no-unresolved rule for packages using exports field instead of main. At this moment, https://github.com/import-js/eslint-plugin-import/blob/main/resolvers/node/index.js only supports main and (non standard) module fields.

@ljharb
Copy link
Member

ljharb commented Jun 18, 2022

This will be done automatically once the resolve package supports exports. It won’t be done until then.

@ljharb ljharb closed this as not planned Won't fix, can't repro, duplicate, stale Jun 18, 2022
@piranna
Copy link
Author

piranna commented Jun 18, 2022

At jest it was needed to be added explicitly on Jest v28 after one year and a half waiting it to be added to resolve package, seems it's abandoned...

@ljharb
Copy link
Member

ljharb commented Jun 19, 2022

@piranna it's not abandoned (i'm the maintainer), and yes, i'm aware jest decided not to wait. However, once resolve supports it, jest will likely switch back to using it.

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

No branches or pull requests

2 participants