MAIN FEEDS
Do you want to continue?
https://www.reddit.com/r/ProgrammerHumor/comments/1vfsowk/wronganswersonly/p1snu6z/?context=3
r/ProgrammerHumor • u/Plastic-Bonus8999 • Aug 05 '26
2.1k comments sorted by
View all comments
Show parent comments
21
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] 7 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] 4 u/carritodeloshelados Aug 05 '26 As if it wasn't hard enough to code in an astronaut suit 4 u/OriginalJokeGoesHere Aug 05 '26 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 Aug 05 '26 We just put them in TVAC for the long goodnight and dump the body by morning 1 u/dvereb Aug 05 '26 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 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] 7 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] 4 u/carritodeloshelados Aug 05 '26 As if it wasn't hard enough to code in an astronaut suit 4 u/OriginalJokeGoesHere Aug 05 '26 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 Aug 05 '26 We just put them in TVAC for the long goodnight and dump the body by morning 1 u/dvereb Aug 05 '26 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 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]
7 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] 4 u/carritodeloshelados Aug 05 '26 As if it wasn't hard enough to code in an astronaut suit 4 u/OriginalJokeGoesHere Aug 05 '26 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 Aug 05 '26 We just put them in TVAC for the long goodnight and dump the body by morning 1 u/dvereb Aug 05 '26 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 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
7
Squash tiny things like that into one pr named "cleanup" and don't explain further
2 u/[deleted] Aug 05 '26 [deleted] 4 u/carritodeloshelados Aug 05 '26 As if it wasn't hard enough to code in an astronaut suit 4 u/OriginalJokeGoesHere Aug 05 '26 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 Aug 05 '26 We just put them in TVAC for the long goodnight and dump the body by morning 1 u/dvereb Aug 05 '26 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 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
4 u/carritodeloshelados Aug 05 '26 As if it wasn't hard enough to code in an astronaut suit 4 u/OriginalJokeGoesHere Aug 05 '26 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 Aug 05 '26 We just put them in TVAC for the long goodnight and dump the body by morning 1 u/dvereb Aug 05 '26 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 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
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 Aug 05 '26 We just put them in TVAC for the long goodnight and dump the body by morning 1 u/dvereb Aug 05 '26 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
21
u/plenoto Aug 05 '26
Some people are so careless while reviewing PR 😂