import-js/eslint-plugin-import

[import/default]: false negative when importing CJS module which uses dynamic import

Open

#2.769 geöffnet am 27. Apr. 2023

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

Repository-Metriken

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

Beschreibung

I found a weird problem with the import/default rule. See this minimal gist

I import two modules which are very similar, both being CJS with default export.

import ecmaScriptLiteralLoader from 'rdf-loader-code/ecmaScriptLiteral.js'
import ecmaScriptModuleLoader from 'rdf-loader-code/ecmaScriptModule.js'

The first is fine but the second triggers a rule

3:8  error  No default export found in imported module "rdf-loader-code/ecmaScriptModule.js"  import/default

Contributor Guide

Research-Richtung
Untersuche die `import/default` Regel im eslint plugin import Repository. Der Fehler tritt auf, wenn ein CJS Modul mit einem Standardexport auch dynamische Importe verwendet, was zu einem falschen Negativ führt. Überprüfe die Logik der Regel für die Handhabung von CJS Modulen, insbesondere wie sie Standardexporte bestimmt. Der Reproduktions Gist unter https://gist.github.com/tpluscode/50d638600e4397a0bef4b3289cfb2925 bietet minimale Testfälle. Suche nach vorhandenen Kommentaren zum Issue, die mögliche Ursachen oder verwandten Code diskutieren.
Tech Stack
javascriptnodejs
Domain
toolingbackend
Issue Type
Fehler
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.
3-5 Tage
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.
Meist klar
Voraussetzungen
Node.jsESLintnpm
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60