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