In code reviews they are whining about formatting with spaces and newlines and naming.

The fact that they are too bored to write unit tests and code coverage cannot reach 20%? Isn't it just sad?

While I agree it might not be the best thing to be talking about right now, a consistent code style can only help a group of developers.

Formatting is one of the first things we hit in code reviews. Then we go through and inspect the code. Then we test it, both unit and otherwise. Then we throw it at the systems guys. And we still have shit slip through. Pays to be thorough.

Time to add code styling tools to your development kit!

I'm thinking about code sniffer, Java checkstyle and jslint and others for other languages

Thanks for the *VERY* helpful comments. The fact that you are still suggesting how to fix the codestyle.... I never came up with the idea to Google it.

But still no one cares for 20% code coverage.

@madburger all I'm saying is that they'd have less occasions to bitch about it if the codee was autoformatted on its way to your CI. It would also be a great occasion to sneakily add merge blocking Belle a certain coverage threshold

@paulwillyjean we are working on it