Skip to content
This repository has been archived by the owner on Jun 17, 2019. It is now read-only.

add eslint-plugin-import/first #107

Open
FrancescoCioria opened this issue Feb 23, 2017 · 0 comments
Open

add eslint-plugin-import/first #107

FrancescoCioria opened this issue Feb 23, 2017 · 0 comments

Comments

@FrancescoCioria
Copy link
Contributor

requirements

From their doc

Notably, imports are hoisted, which means the imported modules will be evaluated before any of the statements interspersed between them. Keeping all imports together at the top of the file may prevent surprises resulting from this part of the spec.

This is currently happening in every project we have that uses:

  • babel6
  • metarpheus
  • prelude

In these projects we call the prelude function before metarpheus import, but since babel6 that import is (correctly) hoisted to the top so the prelude is actually called after it leading to some defects

specs

{optional: describe technical specs to implement this feature, if not obvious}

misc

{optional: other useful info}

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant