209
10000 line PR? LGTM, lol
(i.imgflip.com)
If it compiles it works, right?
I'm not gonna act like I read it all.
you are viewing a single comment's thread
view the rest of the comments
view the rest of the comments
this post was submitted on 11 Feb 2025
209 points (98.2% liked)
Programmer Humor
22189 readers
2765 users here now
Welcome to Programmer Humor!
This is a place where you can post jokes, memes, humor, etc. related to programming!
For sharing awful code theres also Programming Horror.
Rules
- Keep content in english
- No advertisements
- Posts must be related to programming or programmer topics
founded 2 years ago
MODERATORS
The correct response to any PR that is too large to digest is to reject it and ask the author to split it up.
No it is not. It depends on the codebase - if it is something relatively new, a proof of concept or something that is bound to change soon, there is no point in slowing the development down just because it is "too large to digest".
Then saying that you have looked through and reviewed the code would be lying. And that is unprofessional.
Is sitting down and understanding the code in the PR not an option?
I'm not sure what you are getting at.
Clearly they hate efficiencies.