this post was submitted on 03 Nov 2023
80 points (97.6% liked)

Technology

59329 readers
5071 users here now

This is a most excellent place for technology news and articles.


Our Rules


  1. Follow the lemmy.world rules.
  2. Only tech related content.
  3. Be excellent to each another!
  4. Mod approved content bots can post up to 10 articles per day.
  5. Threads asking for personal tech support may be deleted.
  6. Politics threads may be removed.
  7. No memes allowed as posts, OK to post as comments.
  8. Only approved bots from the list below, to ask if your bot can be added please contact us.
  9. Check for duplicates before posting, duplicates may be removed

Approved Bots


founded 1 year ago
MODERATORS
 

I'll be hanging out here I guess. Good luck to the GH team!

you are viewing a single comment's thread
view the rest of the comments
[–] evatronic@lemm.ee 6 points 1 year ago (1 children)

Not two hours ago I declined a PR because it was a huge change on Friday.

[–] bamboo@lemmy.blahaj.zone 15 points 1 year ago (1 children)

Why not just wait to merge it Monday?

[–] evatronic@lemm.ee 9 points 1 year ago (1 children)

'cause stupid corporate policy and metrics mean an unreviewed PR is bad for me, and an unmerged PR is bad for the submitter.

[–] bamboo@lemmy.blahaj.zone 5 points 1 year ago (1 children)

You'd think declined PRs would be more of a metric to deter bad PRs from happening?

[–] Appoxo@lemmy.dbzer0.com 8 points 1 year ago

This would be a logical step and we don't do that here.
Please stop being logical.