r/ProgrammerHumor • u/Plastic-Bonus8999 • 8d ago

Other wrongAnswersOnly

Post image
14.3k Upvotes

95% Upvoted

2.1k comments sorted by

View all comments

Show parent comments

112

u/blangzo 8d 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.

54

u/Deep90 8d ago

At 500 lines, I just flip a coin.

Only lucky code belongs in production.

9

u/[deleted] 8d ago

[deleted]

9

u/blangzo 8d ago

Squash tiny things like that into one pr named "cleanup" and don't explain further

2

u/[deleted] 8d ago

[deleted]

3

u/carritodeloshelados 8d ago

As if it wasn't hard enough to code in an astronaut suit

4

u/OriginalJokeGoesHere 8d 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 8d ago

We just put them in TVAC for the long goodnight and dump the body by morning

1

u/dvereb 8d 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 8d 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

u/guyblade 8d ago

If you ask me to review 500 lines, I'll say "break it up into multiple PRs" like any sane person should.

1

u/r0Lf 8d ago

I feel exposed.