If/else statements always begin on new line.

[LukeAskew]

Consider the following for inclusion:

• if/else if/else statements should always begin on a new line

  // bad
  if (expresion) {
    // ...stuff...
  } else {
    // ...stuff...    
  }
  
  // good
  if (expresion) {
    // ...stuff...
  } 
  else {
    // ...stuff...    
  }

[LukeAskew]

Eric, what are your thoughts on this?

I prefer this style because makes comments on conditional logic more readable.

[LukeAskew]

@kevinmack18 feel free to weigh-in as well.

[nicetransition]

I prefer, but I can learn to change... To me it reads better but I have no reason why I think this...

if (expresion) { // ...stuff... } else { // ...stuff...
}

[mcgaryes]

I do like that. And I agree it is a bit more legible. It will take some getting used to though... And my fear is that you'll lose this styling if you try and run the code through any type of beautified out there... You would essentially have to write your own version for sublime as well as grunt. If you want it to hold through the beautifying process that's Is.

[nicetransition]

I agre with that, @mcgaryes.

[LukeAskew]

I will add the rules, but we should investigate ways to auto-format code that conforms to these standards.

Code Painter seems like it may be a good solution. You guys ever use it?

[nicetransition]

I have not heard of it... but sounds awesome. Have you used it?

[mcgaryes]

Nope never used that before... I've never written a plugin for sublime text 2 either.

If someone wants to take a stab at writing a plugin that works with code painter there's a tutorial on net tuts that looks promising. You'll have to bundle the cod painter executable with the plugin I'd imagine. http://net.tutsplus.com/tutorials/python-tutorials/how-to-create-a-sublime-text-2-plugin/

The grunt task would be easy enough to write.

On Apr 2, 2013, at 10:15 PM, LukeAskew notifications@github.com wrote:

I will add the rules, but we should investigate ways to auto-format code that conforms to these standards.

Code Painter seems like it may be a good solution. You guys ever use it?

— Reply to this email directly or view it on GitHub.