dhruv3006@lemmy.world to Programmer Humor@programming.dev · 5 days agoAmazon AIlemmy.worldimagemessage-square85linkfedilinkarrow-up1848arrow-down16
arrow-up1842arrow-down1imageAmazon AIlemmy.worlddhruv3006@lemmy.world to Programmer Humor@programming.dev · 5 days agomessage-square85linkfedilink
minus-squareEthan@programming.devlinkfedilinkEnglisharrow-up5·5 days agoPR size is an awful metric. The bigger the PR, the less reviewable it is.
minus-squarePoolloverNathan@programming.devlinkfedilinkarrow-up5·5 days agoYes, that’s what the comment said — smaller PRs are better.
minus-squareEthan@programming.devlinkfedilinkEnglisharrow-up4·5 days agoPR size is still an awful metric. It should be within bounds but it should never be an actual metric. Treating it as a metric is an idiotic idea.
PR size is an awful metric. The bigger the PR, the less reviewable it is.
Yes, that’s what the comment said — smaller PRs are better.
PR size is still an awful metric. It should be within bounds but it should never be an actual metric. Treating it as a metric is an idiotic idea.