import-js/eslint-plugin-import

`no-commonjs` does not appear to work for TypeScript

Open

#1 709 ouverte le 30 mars 2020

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)JavaScript (1 540 forks)batch import
help wantedtypescript

Métriques du dépôt

Stars
 (4 946 stars)
Métriques de merge PR
 (Merge moyen 138j 22h) (3 PRs mergées en 30 j)

Description

I have the following rule enabled:

    // disallow require()
    // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-commonjs.md
    'import/no-commonjs': 'error',

However, in my TypeScript source file, the following line that I expected to error does not:

import Boom = require('boom')

Should it?

Guide contributeur

Direction de recherche
Vérifiez l'implémentation de la règle no commonjs pour voir si elle gère la syntaxe import = require() de TypeScript. Assurez vous que l'analyseur TypeScript est utilisé et que la détection du type de nœud AST de la règle couvre cette syntaxe.
Stack technique
javascripttypescript
Domaine
tooling
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 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ESLintTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70