dhruv3006@lemmy.world to Programmer Humor@programming.dev · 4 days agoAmazon AIlemmy.worldimagemessage-square82linkfedilinkarrow-up1838arrow-down16
arrow-up1832arrow-down1imageAmazon AIlemmy.worlddhruv3006@lemmy.world to Programmer Humor@programming.dev · 4 days agomessage-square82linkfedilink
minus-squareEthan@programming.devlinkfedilinkEnglisharrow-up5·3 days agoPR size is an awful metric. The bigger the PR, the less reviewable it is.
minus-squarePoolloverNathan@programming.devlinkfedilinkarrow-up5·3 days agoYes, that’s what the comment said — smaller PRs are better.
minus-squareEthan@programming.devlinkfedilinkEnglisharrow-up4·3 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.