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