Add some core-module tests for no-extraneous-dependencies #644
Conversation
| settings: { 'import/core-modules': ['@namespaced/module'] }, | ||
| }), | ||
| test({ | ||
| code: 'import "@namespaced/module/and/sub/path"', |
There was a problem hiding this comment.
I'm not sure if this is supposed to work, but I was trying to use core-modules to fix the no-extraneous-dependencies for my webpack resolve.alias that points @namespaced/module/lib/foo to ./src/foo
Maybe related to #479 ?
There was a problem hiding this comment.
Ah, weird. Yeah, #479 is the ideal path to resolving this issue without explicit config (Webpack resolver would report as a project or internal) but what you've done should theoretically work.
Must be either the @ or / killing it. I suspect the latter...
|
Rebased this. Part of the issue was that modules have to resolve in order to be reported, and "core-module", for example, didn't exist on disk. I removed one of the namespace tests because |
ljharb
force-pushed
the
extraneous-deps-test-case
branch
from
d93afa0 to
fe51583
Compare
1 similar comment
Added some test cases (one of which fails) for core-modules in no-extraneous-dependencies