import-js/eslint-plugin-import

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

Aperta

#2530 aperta il 23 ago 2022

Vedi su GitHubGuida contributor
 (6 commenti) (0 reazioni) (0 assegnatari)JavaScript (1549 fork)batch import
help wantedtypescript

Metriche repository

Star
 (5940 stelle)
Metriche merge PR
 (Merge medio 138g 22h) (3 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esplora il codice esistente della regola `import/extensions` per capire come gestisce le dichiarazioni di importazione, quindi aggiungi una nuova opzione (ad esempio `typeOnlyImports`) per imporre le estensioni per le importazioni solo di tipo. Guarda il file di test per vedere come vengono testate le opzioni esistenti.
Tech stack
javascripttypescript
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
ESLintTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60