import-js/eslint-plugin-import

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

Open

#919 aberto em 22 de ago. de 2017

Ver no GitHub
 (1 comment) (1 reaction) (0 assignees)JavaScript (1.540 forks)batch import
acceptedhelp wantedrule proposal

Métricas do repositório

Stars
 (4.946 stars)
Métricas de merge de PR
 (Mesclagem média 138d 22h) (3 fundiu PRs em 30d)

Description

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.

Guia do colaborador