import-js/eslint-plugin-import

[import/extensions] Option to enforce extensions for type-only imports

Ouverte

#2 530 ouverte le 23 août 2022

Voir sur GitHubGuide contributeur
 (6 commentaires) (0 réaction) (0 personne assignée)JavaScript (1 548 forks)batch import
help wantedtypescript

Métriques du dépôt

Stars
 (5 940 étoiles)
Métriques de merge PR
 (Merge moyen 138j 22h) (3 PRs mergées en 30 j)

Description

We have a use case for enforcing the .js extension even on type-only imports (I'm aware this is odd).

This requirement stems from supporting TS users using modules (by specifying "type": "module" in their package.json) and specifying nodenext for moduleResolution options in their tsconfig.

Some context can be gleaned from this PR, where I created a failing test using ^this configuration and proceeded to resolve the errors by adding .js extensions to type-only imports. https://github.com/apollographql/apollo-server/pull/6731

Related: https://github.com/import-js/eslint-plugin-import/pull/2270

Does another option seem reasonable to enforce extensions on type-only imports as well? I'd be happy to open a PR to introduce this if so.

Guide contributeur

Direction de recherche
Explorez le code existant de la règle `import/extensions` pour comprendre comment elle traite les instructions d'importation, puis ajoutez une nouvelle option (par exemple `typeOnlyImports`) pour imposer des extensions pour les importations de type uniquement. Regardez le fichier de test pour voir comment les options existantes sont testées.
Stack technique
javascripttypescript
Domaine
backend
Type d'issue
Fonctionnalité
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.
Plutôt claire
Prérequis
ESLintTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60