Rule Proposal: Spacing in `if` statement condition

Not sure if there’s already an ESLint rule for this, I couldn’t find one.

When does this rule warn? Please describe and show example code: Bad:

if (someLongFoo &&
  someLongBar) {

Good:

if (something) {

if (
 someLongFoo &&
 someLongBar
) 

Is this rule preventing an error or is it stylistic? Stylistic

Why is this rule a candidate for inclusion instead of creating a custom rule? It seems like it’d be just as useful as other stylistic rules included with ESLint. Coding guidelines commonly include information on correct spacing for if and switch statements.

Are you willing to create the rule yourself? Yes, if I get a chance to. Happy for someone else to pick it up though.

[Daniel15]

have you ever been able to enforce this in a different tool like JSCS or JSHint? Thanks!

I haven’t. Currently I just enforce it in code reviews.

Would you mind looking at the rule change template and adding any missing information to your post? Thanks!

@platinumazure - Which parts of that template are required for new rules? It looks like the template is mainly for changes to existing rules.

It looks like we’re currently on ESLint 2.7.0.

[kaicataldo]

I’m closing this issue because it looks like consensus couldn’t be reached. While we wish we could accommodate all requests, we have limited resources and do need to prioritize. We’ve found that issues failing to reach consensus after 21 days tend to never reach consensus, and as a team have decided to close such issues. This doesn’t mean the idea isn’t interesting or valuable, only that it’s not something the team can commit to at the moment.