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.

Sign up for GitHub

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

Jump to bottom

Oxlint: Support eslint-plugin-import-x rules #8779

Closed
NateRadebaugh opened this issue Jan 29, 2025 · 1 comment · Fixed by #9074
Closed

Oxlint: Support eslint-plugin-import-x rules #8779

NateRadebaugh opened this issue Jan 29, 2025 · 1 comment · Fixed by #9074
Labels
A-linter Area - Linter C-enhancement Category - New feature or request

Comments

@NateRadebaugh
Copy link

We have eslint-plugin-import-x which touts itself to have the similar set of rules but more performant than eslint-plugin-import

https://github.com/un-ts/eslint-plugin-import-x

Would it make sense for this plugin to also support or map over those eslint rules to the corresponding rules in oxlint?
@NateRadebaugh NateRadebaugh added the C-enhancement Category - New feature or request label Jan 29, 2025
@NateRadebaugh NateRadebaugh mentioned this issue Jan 29, 2025
Closed
@Sysix Sysix added the A-linter Area - Linter label Jan 31, 2025
@Sysix Sysix mentioned this issue Feb 12, 2025
Merged
graphite-app bot pushed a commit that referenced this issue Feb 18, 2025
@Sysix
feat(linter): support import-x plugin name (#9074)
542bbd7 
closes #8779

The `import-x` rules are the reimplementation of the `import` rules. No rules added or removed.
Some eslint-plugins/configs are handling the plugin the same way.

Here a snippet from a config which `next-auth` is using:
https://github.com/antfu/eslint-config/blob/998887cedb0b09c5b95d08602689a2c86e6662c7/src/factory.ts#L56-L67

Could not find more places where some mapping is happening. Please have a look because I did not test it.
@Sysix
Copy link
Collaborator

Sysix commented Feb 18, 2025

will be supported in the next release 🥳

@Sysix Sysix closed this as completed Feb 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-linter Area - Linter C-enhancement Category - New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(linter): support import-x plugin name oxc-project/oxc
2 participants
@NateRadebaugh @Sysix