import-js/eslint-plugin-import

regression: no-extraneous-dependencies when lint target file in a module sub-folder containing package.json

Open

#1.270 geöffnet am 25. Jan. 2019

Auf GitHub ansehen
 (2 Kommentare) (1 Reaktion) (0 zugewiesene Personen)JavaScript (1.540 Forks)batch import
bughelp wanted

Repository-Metriken

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

Beschreibung

Using node@10.15.0 on OSX. Upgraded eslint-plugin-import from 2.14.0 to 2.15.0. Have a package.json that includes numerous dependencies of the form:

"xyz": "git+ssh://git@github.home.247-inc.net:platform/xyz.git#0.0.8",

When running eslint@5.12.1 I now get errors of the following form:

error 'xyz' should be listed in the project's dependencies. Run 'npm i -S xyz' to add it import/no-extraneous-dependencies

Reverting to 2.14.0, the errors are no longer reported.

Contributor Guide

Research-Richtung
Untersuche die Regression von v2.14 zu v2.15 in der no extraneous dependencies Regel. Das Problem ist, dass git+ssh Abhängigkeiten fälschlicherweise als fehlend markiert werden. Sieh dir die Regelquelle in src/rules/no extraneous dependencies.js und die Modulauflösungs Hilfsprogramme an. Überprüfe das Changelog oder den Diff zwischen den Versionen, um den relevanten Commit zu identifizieren. Teste mit einer package.json, die git Abhängigkeiten enthält, um die Parsing Logik zu verstehen.
Tech Stack
javascript
Domain
tooling
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
Node.jsnpm
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70