maccam912@programming.dev to Technology@lemmy.worldEnglish · 11 months agoGithub is having some issueswww.githubstatus.comexternal-linkmessage-square17fedilinkarrow-up182arrow-down12file-text
arrow-up180arrow-down1external-linkGithub is having some issueswww.githubstatus.commaccam912@programming.dev to Technology@lemmy.worldEnglish · 11 months agomessage-square17fedilinkfile-text
minus-squareevatronic@lemm.eelinkfedilinkEnglisharrow-up8arrow-down3·11 months agoNot two hours ago I declined a PR because it was a huge change on Friday.
minus-squarebamboo@lemmy.blahaj.zonelinkfedilinkEnglisharrow-up15·11 months agoWhy not just wait to merge it Monday?
minus-squareevatronic@lemm.eelinkfedilinkEnglisharrow-up10arrow-down1·11 months ago'cause stupid corporate policy and metrics mean an unreviewed PR is bad for me, and an unmerged PR is bad for the submitter.
minus-squarebamboo@lemmy.blahaj.zonelinkfedilinkEnglisharrow-up5·11 months agoYou’d think declined PRs would be more of a metric to deter bad PRs from happening?
minus-squareAppoxo@lemmy.dbzer0.comlinkfedilinkEnglisharrow-up8·11 months agoThis would be a logical step and we don’t do that here. Please stop being logical.
Not two hours ago I declined a PR because it was a huge change on Friday.
Why not just wait to merge it Monday?
'cause stupid corporate policy and metrics mean an unreviewed PR is bad for me, and an unmerged PR is bad for the submitter.
You’d think declined PRs would be more of a metric to deter bad PRs from happening?
This would be a logical step and we don’t do that here.
Please stop being logical.