import-js/eslint-plugin-import

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

Open

#1 654 ouverte le 11 févr. 2020

Voir sur GitHub
 (5 commentaires) (1 réaction) (0 assignés)JavaScript (4 946 stars) (1 540 forks)batch import
bughelp wantedtypescript

Description

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

Guide contributeur

Stack technique
javascripttypescriptnodejs
Domaine
developer experiencetooling
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
ESLint plugin developmentJavaScriptTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
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.