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