import-js/eslint-plugin-import

[import/newline-after-import] false positive with typescript import = require

Open

#1.654 geöffnet am 11. Feb. 2020

Auf GitHub ansehen
 (5 Kommentare) (1 Reaktion) (0 zugewiesene Personen)JavaScript (4.946 Stars) (1.540 Forks)batch import
bughelp wantedtypescript

Beschreibung

When using import/newline-after-import with typescript import = require(..) I get a false positive requiring me to put a newline in the middle of the imports.

import foo from "foo";
import bar = require("bar");
import baz from "baz";

Expected results

No errors

Actual results

1:1 error Expected 1 empty line after import statement not followed by another import import/newline-after-import

Contributor Guide

Tech Stack
javascripttypescriptnodejs
Domain
developer experiencetooling
Issue Type
bug
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 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
ESLint plugin developmentJavaScriptTypeScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
Investigate the `newline after import` rule in the eslint plugin import source code (likely in `src/rules/newline after import.js`). The rule currently does not recognize TypeScript's `import = require()` as an import statement, causing a false positive. Look for where import statements are identified via AST nodes and add support for `TSImportEqualsDeclaration` (or similar) node type. Check if there are existing tests for TypeScript and add a test case.