import-js/eslint-plugin-import

Proposal: Warn on importing from a filename for which a directory also exists

Open

#919 aperta il 22 ago 2017

Vedi su GitHub
 (1 commento) (1 reazione) (0 assegnatari)JavaScript (1540 fork)batch import
acceptedhelp wantedrule proposal

Metriche repository

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

Descrizione

For example, let's say I have the following files:

| something/
|-- index.js
| something.js
| main.js

Inside main.js, I have the following code:

import { helpfulFunction } from './something';

// do something with helpfulFunction()

This is a little dangerous, because did I mean ./something or ./something/? I don't think code should be split up like this—in this case, I would take the contents of something.js and move them to something/index.js and just import from there. A warning would be helpful to track down cases like this.

Guida contributor

Direzione di ricerca
Implementa una nuova regola ESLint che avvisa quando un modulo risolve in un file che ha una directory sorella con lo stesso nome, causando ambiguità.
Tech stack
javascript
Dominio
tooling
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.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75