3 ms·

So if they say "idk Claude did it", what would you write in the PR review box?

by xgulfie 4mo ago

Geezus_42 4mo ago
REJECTED: Engineer does not understand what they wrote.
- manyatoms 4mo ago
  Feels like the title of a blog post someone will write
  - socksy 4mo ago
    Ah, like this one? https://crabby-rathbun.github.io/mjrathbun-website/blog/posts/2026-02-11-gatekeeping-in-open-source-the-scott-shambaugh-story.html https://crabby-rathbun.github.io/mjrathbun-website/blog/post...
    - Geezus_42 4mo ago
      If he did so much verification to prove it's right, why not just write it in the first place?
- dormento 4mo ago
  > Engineer does not understand what they wrote. """""wrote"""""
  - Geezus_42 4mo ago
    s/wrote/pushed/
CursedSilicon 4mo ago
A teammate that can't write (or at least, can't explain) "their own code" Actively drags down the morale and productivity of their team (because everyone is getting flooded with AI slop PR's) AND costs far too much money relative to everyone else doing actual work? (token usage) By god they sound like management material
matkoniecz 4mo ago
"Author of this pull request has not yet reviewed code and does not understand it. This PR was submitted prematurely, probably by accident. Please, check whether you accidentally submitted other unreviewed code - and close such PRs for now and reopen once reviewed."
- maccard 4mo ago
  Don’t ever write this in a professional environment. It’s childish ant achieves nothing other than pissing off the person it’s targeted at and probably the manager who now has to deal with a shitty behaviour complaint.
maccard 4mo ago
The same as if they said it was copied from stack overflow, or if it’s wrong; “I think there’s a problem here, it’s XYZ”. If your peer ignores you and you were wrong, it was their call to make. If you were right - take it to them or the manager depending on how many times it’s happened.

3 ms·

So if they say "idk Claude did it", what would you write in the PR review box?

by xgulfie 4mo ago

So if they say "idk Claude did it", what would you write in the PR review box?

Geezus_42 4mo ago
REJECTED: Engineer does not understand what they wrote.
- manyatoms 4mo ago
  Feels like the title of a blog post someone will write
  - socksy 4mo ago
    Ah, like this one? https://crabby-rathbun.github.io/mjrathbun-website/blog/posts/2026-02-11-gatekeeping-in-open-source-the-scott-shambaugh-story.html https://crabby-rathbun.github.io/mjrathbun-website/blog/post...
    - Geezus_42 4mo ago
      If he did so much verification to prove it's right, why not just write it in the first place?
- dormento 4mo ago
  > Engineer does not understand what they wrote. """""wrote"""""
  - Geezus_42 4mo ago
    s/wrote/pushed/
CursedSilicon 4mo ago
A teammate that can't write (or at least, can't explain) "their own code" Actively drags down the morale and productivity of their team (because everyone is getting flooded with AI slop PR's) AND costs far too much money relative to everyone else doing actual work? (token usage) By god they sound like management material
matkoniecz 4mo ago
"Author of this pull request has not yet reviewed code and does not understand it. This PR was submitted prematurely, probably by accident. Please, check whether you accidentally submitted other unreviewed code - and close such PRs for now and reopen once reviewed."
- maccard 4mo ago
  Don’t ever write this in a professional environment. It’s childish ant achieves nothing other than pissing off the person it’s targeted at and probably the manager who now has to deal with a shitty behaviour complaint.
maccard 4mo ago
The same as if they said it was copied from stack overflow, or if it’s wrong; “I think there’s a problem here, it’s XYZ”. If your peer ignores you and you were wrong, it was their call to make. If you were right - take it to them or the manager depending on how many times it’s happened.