finn-no / eslint-config-finn

ESLint config for Finn.no
2 stars 3 forks source link

Allow hoisting functions [no-use-before-define] #8

Closed gregersrygg closed 8 years ago

gregersrygg commented 8 years ago

Almost what we agreed on in the code-style meeting.

Reason: It's safe to rely on hoisting of function declarations, so it doesn't make sense to us to disallow it. We also talked about allowing hoisting of classes, but according to the eslint docs, it's not safe - so I vote for disallowing hoisting of classes. http://eslint.org/docs/rules/no-use-before-define