Early exit vs if else

[dcousens]

Case 1

if (condition) {
  return true
} else {
  // ... code
}

Case 2

if (condition) return true

// ... code

IMHO, I find case 2 to be the clearest and cleanest, @feross did you have any thoughts on this? Should we align to either convention? Even if we do, can we do anything about it?

It would have to be consistent for continue, break and return statements.

[feross]

I prefer case 2, but there's no way to enforce this with eslint, nor do I think we should do that. :)

[dcousens]

No worries, just though I'd bring it up as I had a case of nit picking about it in a recent repository, and it just stirred up an unnecessary discussion.