Found multiple import of the same path JS-R1000
Style
Minor
15 days ago
—
15 days old
'/code/src/app/index.ts' imported multiple times
19
20<script>
21import { Registry, BareViewManager, ViewManager } from "../app/index";
22import { Examples } from "@/app/index"; 23import ResolutionToolbar from "@/components/ResolutionToolbar";
24import ZoomSlider from "@/components/ZoomSlider";
25import ComponentContextMenu from "@/components/ComponentContextMenu";'/code/src/app/index.ts' imported multiple times
18</template>
19
20<script>
21import { Registry, BareViewManager, ViewManager } from "../app/index"; 22import { Examples } from "@/app/index";
23import ResolutionToolbar from "@/components/ResolutionToolbar";
24import ZoomSlider from "@/components/ZoomSlider";'/code/src/app/core/init.ts' imported multiple times
44import ExportUtils, { SerializationError } from "@/app/utils/exportUtils";
45import { LogicalLayerType, InterchangeV1_2, ValveType } from "@/app/core/init";
46
47import { Point } from "@/app/core/init"; 48
49/**
50 * View manager class'/code/src/app/core/init.ts' imported multiple times
42
43import LoadUtils from "@/app/utils/loadUtils";
44import ExportUtils, { SerializationError } from "@/app/utils/exportUtils";
45import { LogicalLayerType, InterchangeV1_2, ValveType } from "@/app/core/init"; 46
47import { Point } from "@/app/core/init";
48'paper' imported multiple times
1import paper from "paper";
2import {SymbolDefinition} from "paper"; 3import AdaptiveGrid from "../grid/adaptiveGrid";
4
5export default class GridRenderer {
Description
It is always recommended to group all imports of a single path or source in a single import declaration. These usually happen in a codebase when multiple developers are working on the same codebase. It helps improving code refactoring and code readability as well.
Bad Practice
// <!-- index.js -->
import calc from './calc.js'
import { add } from './calc.js'
console.log(add(1, 2))
Recommended
// <!-- index.js -->
import calc, { add } from './calc.js'
console.log(add(1, 2))