I honestly wish for a PR this size. One of the ones that came across this week was 813 commits, +17K -2K.
Of the 250 commits that GitHub was willing to show it had 35 other PRs merged into this massive one. Why they thought one giant PR was somehow better I’ll never know.
Of course…high priority, please review and merge immediately. Like guys it’s gonna take me a week to make sense of this.
This is a joke I’m not college educated enough to understand.
It’s basically saying “insert one ass loads of code in one go, all written by AI”.
That’s an automatic reject from me, that’s not a patch, it’s an overhaul
No
You ask your LLM of choice to look it over, completing the shit-cycle
🛑 Changes requested
Too big. Break into smaller individual PRs.
Repeat after me: “Rejected. Reason: too large of a change for one PR.”
If you can’t be bothered to write your own code, I can’t be bothered to review
The problem with Claude is that it doesn’t write code to be modular & reusable. Every tiny change requires a complete rewrite.
I’ve completely banned any code that can’t be explained. I’ve had my CTO send me code at 3 AM to implement and when I ask him what I’m looking at he just says it doesn’t need review, just push it.
Uhh, no sir, I’m not doing shit because you’ve handed me GCC and we’re MSVC.
After I bitched endlessly to the CEO about that he said I have final say on what goes into the project.
I’ve had my CTO send me code at 3 AM
I hope you don’t even respond until your next normal working hours!
I love my job, even when I have to deal with nonsense like that and I’m compensated very well to be on call 24/7.
No amount of money would make me put my health at risk like that. Been there at a job before where I was always working. No thanks.
that sets a really bad example. you shouldn’t do that to yourself and you shouldn’t allow it to happen to anyone else.
He knows best what’s best for him. If he’s explicitly paid extra to be on call 24/7, and he’s happy with that extra. Let him be om-call 24/7.
There are situations and jobs where 24/7 availability is needed. Someone has to do it. And if that someone believes he’s getting enough of a compensation for it, there’s nothing wrong with it.
well, that depends entirely on your prompt/process, it can be done
It can but I’m not looking to make things even more complicated. We have enough unexplained non-sense in the project as is, having to link it correctly is just endless pain that I don’t want to deal with.
With enough seniour developer’s time and dedication you can spend days and enough water to flood a town, so you can badly maybe do something that a junior dev can do already (your shit will still be worse). If that’s not an achievement of a modern technology I don’t know what is.
I struggle to review a 1k line change. When people give me such big changes I normally don’t believe they’ve reviewed them either.
That’s because they haven’t.
Mystery solved!
Try working on a codebase that’s all event-driven hexagonal CQRS with hand-crafted SQL for persistence. Add additional buzzwordy methodologies to taste.
Adding a single property to your product means you now have to update an aggregate class, several DTOs, and several event classes and handlers before you can even think about touching the UI.
And that’s in your main solution. There’s also at least one facade service you’ll need to make compatible and you also need to update the event simulator used for testing. The latter night involve having to touch every single line in a 2000 lines long SQL script.
Having to go though three separate 600-2000 LOC PRs for one PBI isn’t that exotic.
forgive them lord, for they know not what they do
deleted by creator
Occasionally I do that…but only because 500 of those lines are my comments explaining everything.
Don’t worry half of those will be useless code comments
// Here I'm not using that other thing that is now completely irrelevant, but I'll leave a comment to the non-existing thing anyway because I'm avoiding it.REEEEEEEEEEEEEEEEEEEEEEEEEEEEEEE
The comment:
# This code does exactly what you asked: Never change state, only fetch the state and return the difference
the code: hallucinated database table drops
It seems you have a friend named Claude
It is not even peer review anymore, unless we are pretending that claude is our peer.
LGTM
Let’s Gamble, Try Merging is my favourite
Let’s Go Topple the Monarchy!
And people were already annoyed when I had ~90 changes due to refactoring and fixing imports …
Peer review? Aah you’re referring to the Math proof Astra did the other day.
This is why I have my agent do a “cleanup” pass to trim down as much redundancy as possible, tidy up the comments, and so forth. Ideally break the change into several independent changelists,
This one weird trick will blow your mind: Just add “do it correctly” to your prompt!
That doesn’t work so well since it leaves all the old context in place. The point of doing a separate cleanup pass is to get the AI to look at it all holistically with “fresh eyes”.
I guess this is a humor community, though, so practical advice is getting downvoted. Um… bazinga? That usually triggers the laugh track.
I find LLMs produce the best output when I give them a stupid backstory to set the mood:
The inspiration for this code module is “I’m hungry but too tired to get off the couch”. You just got home from work and are exhausted and sit down, but can’t get up now. You should have eaten dinner first. You put on the TV and watch something. All channels have been replaced with the cooking channel. Someone is making pizza next door. You sit back and doze off, dreaming of mozzarella mattresses and pepperoni pillows. Eventually you wake up from your nap, just perky enough to put on a microwave pizza. It’s almost as good as you imagined, but you burn yourself again because you’re impatient. Your boss asks you to make [insert code change here]. You procrastinate a bit, but eventually do it just before going to bed.
Are you also using a different model and harness or are you just ignoring the 10k+ tokens of system context that claude code has in its “holistic view”?
I don’t use Claude Code. Until recently I was using Cline, it’s got the ability to switch between different models for exactly this sort of thing. I switched to Qwen Code recently and its model-switching wasn’t as convenient, though it just rolled out an update that makes it much easier. I’ll experiment a bit with that, it might have caught up with Cline in that regard.
Skip a step and have your agent reject the PR for you.
