search

segmentio / eslint-config

Segment's ESLint configurations.
9 stars 6 forks source link

Enable cyclomatic complexity rule #8

Closed ndhoule closed 8 years ago

ndhoule commented 9 years ago

Singling out a.js-integrations as an example, a lot of our functions do way too much. We should enable complexity to prevent this from happening.

http://eslint.org/docs/rules/complexity

Should be a warning at first, and an error down the line. Thinking the setting should be high and we can gradually decrease it.

stephenmathieson commented 8 years ago

+1

dominicbarnes commented 8 years ago

+1 for a warning

ndhoule commented 8 years ago

sweet, let's try it out!