search

import-js / eslint-import-resolver-typescript

This plugin adds `TypeScript` support to `eslint-plugin-import`
https://www.npmjs.com/package/eslint-import-resolver-typescript
ISC License
698 stars 61 forks source link

Allow `import-x` plugin, not only `import` #293

Open RobinTail opened 1 week ago

RobinTail commented 1 week ago

There is a line in package.json:

https://github.com/import-js/eslint-import-resolver-typescript/blob/ca11f1c538714252ff058a8e1c680796ee5775d0/package.json#L66

That causes warning during the installation of this package when using eslint-plugin-import-x:

warning " > eslint-import-resolver-typescript@3.6.1" has unmet peer dependency "eslint-plugin-import@*".

Though, even the Readme file suggest to try it (under its former name):

This plugin adds TypeScript support to eslint-plugin-import (Or maybe you want to try eslint-plugin-i for faster speed)

I'd like to request either marking that dependency as optional peer (using peerDepedenciesMeta entry) or ask for another solution on how these two packages can be installed together without warning.

RobinTail commented 1 week ago

I also found another workaround while this issue is not resolved. Overriding the import plugin with import-x in package.json:

"eslint-plugin-import": "https://registry.npmjs.org/eslint-plugin-import-x/-/eslint-plugin-import-x-0.5.2.tgz",

Though, I admit, it's hacky.