@[email protected] to TechnologyEnglish • 1 year agoGithub is having some issueswww.githubstatus.comexternal-linkmessage-square15fedilinkarrow-up183arrow-down12file-textcross-posted to: [email protected]
arrow-up181arrow-down1external-linkGithub is having some issueswww.githubstatus.com@[email protected] to TechnologyEnglish • 1 year agomessage-square15fedilinkfile-textcross-posted to: [email protected]
minus-square@[email protected]linkfedilinkEnglish6•1 year agoNot two hours ago I declined a PR because it was a huge change on Friday.
minus-square@[email protected]linkfedilinkEnglish9•1 year 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-square@[email protected]linkfedilinkEnglish5•1 year agoYou’d think declined PRs would be more of a metric to deter bad PRs from happening?
minus-square@[email protected]linkfedilinkEnglish8•1 year 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.