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 main to work around eslint-plugin-import issues #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tanhauhau
Copy link
Contributor

No description provided.

@yannbf
Copy link

yannbf commented Feb 14, 2022

This sounds like a great (and needed) feature! Any chance this could get merged? Thanks!

@benmccann
Copy link

import-js/eslint-plugin-import#1810 is still not fixed, but at this point I'm of the opinion that they need to fix their package rather than expecting the entire ecosystem to work around their lack of support for the latest standards. There's been a PR for browserify/resolve#222 open for three years and it's still stuck in limbo. It's getting to be a bit ridiculous

@ljharb
Copy link

ljharb commented Jun 15, 2023

@benmccann again, i'd appreciate if you avoided hostility, especially when describing unpaid volunteer work that's not proceeding as fast as you'd like.

@benmccann
Copy link

I've discovered another solution is that you can also use eslint-import-resolver-exports to configure eslint-plugin-import to correctly handle this case. It's admittedly rather cumbersome to setup, but that's really something that should be addressed in those projects

Another option would be to use tsc rather than eslint-plugin-import. That's what I've done in my own projects as I found that it catches all the same issues

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.

4 participants