DEV Community

Discussion on: Doing code review

Collapse
 
havespacesuit profile image
Eric Sundquist

I like the 500 lines rule. I would be interested if that could be incorporated as an automated rule in a devops setting - with the ability to override of course. It would really help devs to consider the scope of what they are working on when they know they are going to need to defend any huge changes.

Collapse
 
mblayman profile image
Matt Layman

Yeah, that's an interesting idea. I bet someone could whip up a one-liner with diffstat for a PR that could fail a CI build if the branch was too big. Overriding that behavior would be the hard part though.