DEV Community

Discussion on: A Clear, Concise & Comfy Code Review Checklist

Collapse
 
chris_bertrand profile image
Chris Bertrand

A linter to check code duplication would be awesome! Do you know of any?

We all know, there are only two hard things in Computer Science: cache invalidation and naming things. That's why I check naming, even if it is entirely subjective.

Focusing on architectural/design issues at a code review is a little late for me, those should be ironed out at planning, however they should be raised at this stage if that did not occur. Thanks for the comment, I'll add in a section for it.

Collapse
 
joefuzz profile image
Joe DuVall

To add to Aleksei's list, we use Sonarqube to catch everything you mentioned for Java, Javascript, TypeScript, and C#. There are plenty of other plugins that expand Sonarqube's language support, too.

sonarqube.org/

Thread Thread
 
dan1ve profile image
Daniel Veihelmann • Edited

We use Teamscale for this purpose, as it is much faster and independent from the build.
teamscale.com

Thread Thread
 
chris_bertrand profile image
Chris Bertrand

Thanks Dan, Looks good. There are so many different tools! Picking one is always a conundrum.

Thread Thread
 
pravinuttarwar profile image
Pravin Uttarwar

We use codegrip.tech, its automated, supports more than 20+ languages and more importantly has great UI and integrations with slack. Price is very low compare to different tools present in market.

Thread Thread
 
chris_bertrand profile image
Chris Bertrand

Nice, thanks for the introduction. Will take a look.