import-js/eslint-plugin-import

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

Open

#919 opened on Aug 22, 2017

 (1 comment) (1 reaction) (0 assignees)JavaScript (1,548 forks)batch import
acceptedhelp wantedrule proposal

Repository metrics

Stars
 (5,940 stars)
PR merge metrics
 (PR metrics pending)

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.

Contributor guide