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

Technology

59123 readers
2310 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
[–] [email protected] 6 points 1 year ago (1 children)

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

[–] [email protected] 16 points 1 year ago (1 children)

Why not just wait to merge it Monday?

[–] [email protected] 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.

[–] [email protected] 5 points 1 year ago (1 children)

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

[–] [email protected] 8 points 1 year ago

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