import-js/eslint-plugin-import

import/no-duplicates does not catch all duplicates

Open

#1.456 geöffnet am 19. Aug. 2019

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (1.540 Forks)batch import
enhancementflowhelp wanted

Repository-Metriken

Stars
 (4.946 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 138T 22h) (3 gemergte PRs in 30 T)

Beschreibung

Hi, I came across an example in our codebase where this rule did not catch the duplicate import.

import { ProfilePictureCard, SectionCard } from 'our-component-library';
import type { NativeEventHandler } from 'our-component-library';
// Expected ^^ to trigger error, but it did not.

However this is correctly reported:

import { ProfilePictureCard, SectionCard } from 'our-component-library';
import { type NativeEventHandler } from 'our-component-library';
// ^^ Triggers error.

Contributor Guide

Research-Richtung
Untersuche die Implementierung der no duplicates Regel, um zu verstehen, warum sie separate import type Anweisungen nicht markiert. Konzentriere dich darauf, wie die Regel Import Spezifikationen sammelt und vergleicht.
Tech Stack
javascript
Domain
devtools
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70