Nikolay Vitkov
d76576c6fe
This is need to be able to move to flat config which I am handling in a separate CL. The overrides are there as these packages got update today, so we need to way, but they were last touched 3 years ago, so reviewing their changes upstream as well. Bug: none Change-Id: I61534f881fd4e747c716b42b773f8f7f3e9bc761 Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6063618 Commit-Queue: Nikolay Vitkov <nvitkov@chromium.org> Reviewed-by: Danil Somsikov <dsv@chromium.org>
1.8 KiB
import/max-dependencies
Forbid modules to have too many dependencies (import or require statements).
This is a useful rule because a module with too many dependencies is a code smell, and usually indicates the module is doing too much and/or should be broken up into smaller modules.
Importing multiple named exports from a single module will only count once (e.g. import {x, y, z} from './foo' will only count as a single dependency).
Options
This rule has the following options, with these defaults:
"import/max-dependencies": ["error", {
"max": 10,
"ignoreTypeImports": false,
}]
max
This option sets the maximum number of dependencies allowed. Anything over will trigger the rule. Default is 10 if the rule is enabled and no max is specified.
Given a max value of {"max": 2}:
Fail
import a from './a'; // 1
const b = require('./b'); // 2
import c from './c'; // 3 - exceeds max!
Pass
import a from './a'; // 1
const anotherA = require('./a'); // still 1
import {x, y, z} from './foo'; // 2
ignoreTypeImports
Ignores type imports. Type imports are a feature released in TypeScript 3.8, you can read more here. Defaults to false.
Given {"max": 2, "ignoreTypeImports": true}:
Fail
import a from './a';
import b from './b';
import c from './c';
Pass
import a from './a';
import b from './b';
import type c from './c'; // Doesn't count against max
When Not To Use It
If you don't care how many dependencies a module has.