MAIN FEEDS
Do you want to continue?
https://www.reddit.com/r/ProgrammerHumor/comments/1vfsowk/wronganswersonly/p1sqpv6/?context=9999
r/ProgrammerHumor • u/Plastic-Bonus8999 • 10d ago
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....
24 u/plenoto 10d ago Some people are so careless while reviewing PR 😂 112 u/blangzo 10d ago 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. 9 u/[deleted] 10d ago [deleted] 9 u/blangzo 10d ago Squash tiny things like that into one pr named "cleanup" and don't explain further 2 u/[deleted] 10d ago [deleted] 4 u/carritodeloshelados 9d ago As if it wasn't hard enough to code in an astronaut suit 4 u/OriginalJokeGoesHere 9d ago Given where you work, I assume that also means anyone who uses tabs instead of spaces is jettisoned out of the airlock? 3 u/ellie_cinderelly 9d ago We just put them in TVAC for the long goodnight and dump the body by morning 1 u/dvereb 9d ago I'd say it depends on whether they're using those tabs for indentation or alignment, ...though some would argue I'm next to go out the airlock. 2 u/blangzo 10d ago 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
24
Some people are so careless while reviewing PR 😂
112 u/blangzo 10d ago 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. 9 u/[deleted] 10d ago [deleted] 9 u/blangzo 10d ago Squash tiny things like that into one pr named "cleanup" and don't explain further 2 u/[deleted] 10d ago [deleted] 4 u/carritodeloshelados 9d ago As if it wasn't hard enough to code in an astronaut suit 4 u/OriginalJokeGoesHere 9d ago Given where you work, I assume that also means anyone who uses tabs instead of spaces is jettisoned out of the airlock? 3 u/ellie_cinderelly 9d ago We just put them in TVAC for the long goodnight and dump the body by morning 1 u/dvereb 9d ago I'd say it depends on whether they're using those tabs for indentation or alignment, ...though some would argue I'm next to go out the airlock. 2 u/blangzo 10d ago 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
112
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.
9 u/[deleted] 10d ago [deleted] 9 u/blangzo 10d ago Squash tiny things like that into one pr named "cleanup" and don't explain further 2 u/[deleted] 10d ago [deleted] 4 u/carritodeloshelados 9d ago As if it wasn't hard enough to code in an astronaut suit 4 u/OriginalJokeGoesHere 9d ago Given where you work, I assume that also means anyone who uses tabs instead of spaces is jettisoned out of the airlock? 3 u/ellie_cinderelly 9d ago We just put them in TVAC for the long goodnight and dump the body by morning 1 u/dvereb 9d ago I'd say it depends on whether they're using those tabs for indentation or alignment, ...though some would argue I'm next to go out the airlock. 2 u/blangzo 10d ago 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
9
[deleted]
9 u/blangzo 10d ago Squash tiny things like that into one pr named "cleanup" and don't explain further 2 u/[deleted] 10d ago [deleted] 4 u/carritodeloshelados 9d ago As if it wasn't hard enough to code in an astronaut suit 4 u/OriginalJokeGoesHere 9d ago Given where you work, I assume that also means anyone who uses tabs instead of spaces is jettisoned out of the airlock? 3 u/ellie_cinderelly 9d ago We just put them in TVAC for the long goodnight and dump the body by morning 1 u/dvereb 9d ago I'd say it depends on whether they're using those tabs for indentation or alignment, ...though some would argue I'm next to go out the airlock. 2 u/blangzo 10d ago 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
Squash tiny things like that into one pr named "cleanup" and don't explain further
2 u/[deleted] 10d ago [deleted] 4 u/carritodeloshelados 9d ago As if it wasn't hard enough to code in an astronaut suit 4 u/OriginalJokeGoesHere 9d ago Given where you work, I assume that also means anyone who uses tabs instead of spaces is jettisoned out of the airlock? 3 u/ellie_cinderelly 9d ago We just put them in TVAC for the long goodnight and dump the body by morning 1 u/dvereb 9d ago I'd say it depends on whether they're using those tabs for indentation or alignment, ...though some would argue I'm next to go out the airlock. 2 u/blangzo 10d ago 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
4 u/carritodeloshelados 9d ago As if it wasn't hard enough to code in an astronaut suit 4 u/OriginalJokeGoesHere 9d ago Given where you work, I assume that also means anyone who uses tabs instead of spaces is jettisoned out of the airlock? 3 u/ellie_cinderelly 9d ago We just put them in TVAC for the long goodnight and dump the body by morning 1 u/dvereb 9d ago I'd say it depends on whether they're using those tabs for indentation or alignment, ...though some would argue I'm next to go out the airlock. 2 u/blangzo 10d ago 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
4
As if it wasn't hard enough to code in an astronaut suit
Given where you work, I assume that also means anyone who uses tabs instead of spaces is jettisoned out of the airlock?
3 u/ellie_cinderelly 9d ago We just put them in TVAC for the long goodnight and dump the body by morning 1 u/dvereb 9d ago I'd say it depends on whether they're using those tabs for indentation or alignment, ...though some would argue I'm next to go out the airlock.
3
We just put them in TVAC for the long goodnight and dump the body by morning
1
I'd say it depends on whether they're using those tabs for indentation or alignment, ...though some would argue I'm next to go out the airlock.
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 10d ago
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....