import-js/eslint-plugin-import

Check for importing named exports from CJS module

Open

#928 geöffnet am 15. Sept. 2017

Auf GitHub ansehen
 (8 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)JavaScript (1.540 Forks)batch import
enhancementhelp wanted

Repository-Metriken

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

Beschreibung

With the new node.js 8.5 ESM support importing CJS modules as if they had named exports is not supported. (Unlike Babel)

Eg:

// bad
import { get } from 'lodash';

//good
import _ from 'lodash';
const get = _.get;

//good
import { get } from 'lodash-es';

Contributor Guide

Research-Richtung
Implementiere eine neue Regel, die benannte Importe aus CommonJS Modulen erkennt. Die Regel sollte im Verzeichnis src/rules platziert werden, entsprechend dem Muster bestehender Regeln. Sieh dir ähnliche Regeln wie 'no named as default' als Referenz an. Stelle sicher, dass sowohl statische als auch dynamische Importe behandelt werden, und berücksichtige den Fall, dass das Modul ein 'esModuleInterop' Flag hat oder ein bekanntes CJS Modul wie 'lodash' ist. Die Diskussion des Issues schlägt vor, das 'resolve' Modul zu verwenden, um zu prüfen, ob das importierte Modul CJS ist.
Tech Stack
javascript
Domain
developer experiencetooling
Issue Type
Funktion
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.
Halber Tag
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
JavaScriptESLintNode.js modules
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60