search

JSRocksHQ / harmonic

The next static site generator
http://harmonicjs.com/
MIT License
282 stars 26 forks source link

ESLint config: use the "nofunc" additional option in `no-use-before-define` #172

Closed UltCombo closed 8 years ago

UltCombo commented 9 years ago

This has caused quite a few refactoring hazards as seen here.

We should add the "nofunc" option to the no-use-before-define rule config in order to ignore function declarations.

What do you guys think?

/cc @viniciusdacal @jaydson

viniciusdacal commented 9 years ago

Yeah, I think it's a great idea :smile: . As you said before, a function it's available in the entire scope. I don't think that's is a bad practice, if we look for some "clean code" practices, like the "The Step- down Rule", that kind of situation will happen always.

UltCombo commented 8 years ago

:sweat_smile: This somehow fell off my radar, fixed now.