import-js/eslint-plugin-import

Disallow duplicate require

Open

#2.925 geöffnet am 18. Nov. 2023

Auf GitHub ansehen
 (9 Kommentare) (1 Reaktion) (0 zugewiesene Personen)JavaScript (4.946 Stars) (1.540 Forks)batch import
enhancementhelp wanted

Beschreibung

I have following code :

const { addCellToRow } = require("./cell-utils.js");
const { setCellValue, findCell, createCell } = require("./cell-utils.js");

Both require are using the same path, and should thus be grouped.

I've tried to enable this in my .eslintrc :

"import/no-duplicates": ["error", {"prefer-inline": true}],

But it didn't have any effect.

Are there any rules that can catch this ?

Contributor Guide

Tech Stack
javascriptnodejs
Domain
tooling
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
half day
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
active
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Understanding of ESLint plugin developmentFamiliarity with AST for require statements
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40
Research-Richtung
Investigate the existing `import/no duplicates` rule to understand why it does not catch duplicate `require()` calls. Check the rule's source code in the repository to see how it handles `import` vs `require`. Look at the issue comments for any suggested fixes or workarounds. Consider whether a new rule or an enhancement to the existing rule is needed.