search

greggman / eslint-plugin-one-variable-per-var

Enforce one variable declaration per var statement
2 stars 1 forks source link

Push upstream #1

Open tlvince opened 9 years ago

tlvince commented 9 years ago

I think this rule is generic enough to be contributed back upstream and included in the growing variables section. Worth a try?

greggman commented 9 years ago

I suppose I could try. It seems like most JS programmers prefer one var multiple variables so I kind of assumed they'd be hostile toward it.

DrewML commented 9 years ago

@greggman I don't think (at least I would hope) people aren't hostile towards it.

One of the big selling points of ESLint is that it's configurable for a large variety of projects/teams. For my project, we actually enforce this rule to prevent developers from having issues with certain browserify transforms (https://github.com/substack/brfs/issues/28). In this case, it's not even preference for us - not having that rule would cause problems for future developers.

+1 for contributing this upstream.