Removed useless else after return

[alexmanno]

As a title, I removed the useless else after returns.

(Without changing the logic)

[lagbox]

These are the valueless changes that people have been complaining about.
I can't even remember where this shhtuff stands at the moment, been down this path before.

[alexmanno]

@lagbox That's very welcoming.

[lagbox]

@taylorotwell is this restyling of the framework actually happening or not as this type of stuff seems very inconsistent in what is merged and what isn't? Just trying to get an idea as I wasn't able to follow all the previous discussions on this exact thing.

[browner12]

these PRs are not being accepted. we have had a discussion about improving our Contribution Guide to go over style stuff like this so we can bring more consistency to the framework. I had intended to take this on, but have been swamped lately.

[alexmanno]

Many thanks for explanations @browner12 :)

[lagbox]

@alexmanno Sorry, that was more of a note in general not quite for you, sorry about the tone.
Not valueless as in we don't value the contribution; valueless as in it doesn't actually add any value to the code, its just a term.

There has been an inconsistency with this particular issue over the last couple of months which is why I had thrown that out.

There was a reason (preference and explicit nature of them) for them being kept. Just wanted to know the final position on these.

@browner12 Thanks, I didn't know that was on you.

[browner12]

I had brought it up and volunteered, but if someone wants to go for it, more power to them.

Like I said, I'm swamped at work now, so it's not going to get done soon if it's on me.

[lagbox]

@browner12 maybe we can figure something out, im always on freenode[irc] (lagbox)