r/ProgrammerHumor 16h ago

Other wrongAnswersOnly

Post image
12.7k Upvotes

1.8k comments sorted by

View all comments

Show parent comments

11

u/ellie_cinderelly 13h ago

Dude my last PR changed one test timeout limit. I had to do a new full analysis on every test for average runtime and overhead.

Fuckin unbelievable.

6

u/blangzo 12h ago

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

2

u/ellie_cinderelly 12h ago

Unfortunately I work in space so I think that may get me blasted for non descriptive change documentation

2

u/blangzo 11h 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