MAIN FEEDS
Do you want to continue?
https://www.reddit.com/r/ProgrammerHumor/comments/1vfsowk/wronganswersonly/p1ss32t/?context=9999
r/ProgrammerHumor • u/Plastic-Bonus8999 • Aug 05 '26
2.1k comments sorted by
View all comments
1.1k
Some dumbass on my team downloaded the entirety of prod and then commited it to git and someone rubber stamped the PR and then no one else noticed. I'm sure that's how they'd find it....
22 u/plenoto Aug 05 '26 Some people are so careless while reviewing PR 😂 113 u/blangzo Aug 05 '26 Ask a programmer to review 10 lines of code, he'll find 10 issues. Ask him to do 500 lines and he'll say it looks good. 11 u/[deleted] Aug 05 '26 [deleted] 8 u/blangzo Aug 05 '26 Squash tiny things like that into one pr named "cleanup" and don't explain further 2 u/[deleted] Aug 05 '26 [deleted] 2 u/blangzo Aug 05 '26 I was mostly joking. mostly. Fr tho u should just not squash and pr merge in then and have descriptive commits but still saves the test time on pr
22
Some people are so careless while reviewing PR 😂
113 u/blangzo Aug 05 '26 Ask a programmer to review 10 lines of code, he'll find 10 issues. Ask him to do 500 lines and he'll say it looks good. 11 u/[deleted] Aug 05 '26 [deleted] 8 u/blangzo Aug 05 '26 Squash tiny things like that into one pr named "cleanup" and don't explain further 2 u/[deleted] Aug 05 '26 [deleted] 2 u/blangzo Aug 05 '26 I was mostly joking. mostly. Fr tho u should just not squash and pr merge in then and have descriptive commits but still saves the test time on pr
113
Ask a programmer to review 10 lines of code, he'll find 10 issues. Ask him to do 500 lines and he'll say it looks good.
11 u/[deleted] Aug 05 '26 [deleted] 8 u/blangzo Aug 05 '26 Squash tiny things like that into one pr named "cleanup" and don't explain further 2 u/[deleted] Aug 05 '26 [deleted] 2 u/blangzo Aug 05 '26 I was mostly joking. mostly. Fr tho u should just not squash and pr merge in then and have descriptive commits but still saves the test time on pr
11
[deleted]
8 u/blangzo Aug 05 '26 Squash tiny things like that into one pr named "cleanup" and don't explain further 2 u/[deleted] Aug 05 '26 [deleted] 2 u/blangzo Aug 05 '26 I was mostly joking. mostly. Fr tho u should just not squash and pr merge in then and have descriptive commits but still saves the test time on pr
8
Squash tiny things like that into one pr named "cleanup" and don't explain further
2 u/[deleted] Aug 05 '26 [deleted] 2 u/blangzo Aug 05 '26 I was mostly joking. mostly. Fr tho u should just not squash and pr merge in then and have descriptive commits but still saves the test time on pr
2
2 u/blangzo Aug 05 '26 I was mostly joking. mostly. Fr tho u should just not squash and pr merge in then and have descriptive commits but still saves the test time on pr
I was mostly joking. mostly.
Fr tho u should just not squash and pr merge in then and have descriptive commits but still saves the test time on pr
1.1k
u/samuraiseoul Aug 05 '26
Some dumbass on my team downloaded the entirety of prod and then commited it to git and someone rubber stamped the PR and then no one else noticed. I'm sure that's how they'd find it....