DEV Community

Discussion on: The Pull Request Checklist

Collapse
 
samuraiseoul profile image
Sophie The Lionhart

I think you should check out Codacy. We use it at our current company and I almost love it. I love the concept of it for sure.

It implements code linters and static analyzers on every PR you have so it looks for a lot of the things you have above. Certain things obviously you can't catch with it and need a human touch still, such as the test edge cases though.

Its nice because it allows the person doing the PR to no have to look through all the BS parts of a PR. They only have to focus on things that a linter or analyzer can't catch. Also when it complains that there is a mistake in the PR, it links to education about the mistake so that the developer grows. Its basically free mentoring on the easy stuff, which is the part I love!

I feel there has to be some kind of PR checklist integration for github but I haven't found one. That said I've put in very little effort looking.

The only problem with Codacy is that sometimes it takes a while to get a PR analyzed, like a few hours. Its gotten a lot better recently, but that's one of the pain points.

Collapse
 
idoshamun profile image
Ido Shamun

Thanks! I'll look into it.

Collapse
 
samuraiseoul profile image
Sophie The Lionhart

Yeah, let me know if you end up using it!