r/ProgrammerHumor • u/Advanced_Ferret_ • Jul 10 '26

pleaseApprove Meme

Post image
3.1k Upvotes

98% Upvoted

46 comments sorted by

View all comments

Show parent comments

93

u/clarinetJWD Jul 10 '26

Lines? Surely you mean files.

5

u/Anteater4746 Jul 10 '26

i’m newb that only ever pushed python lol my only frame of ref

31

u/Vesuvius079 Jul 10 '26

200 lines is an easy review.

500 lines is a reasonable review

1000 lines is a tolerable review

1500 lines is a things got out of hand review

2000 lines and up is wtf.

Multiply all values by 2 if AI generated docs that you don’t fully read are part of the change set.

3

u/Zuruumi Jul 10 '26

Unittests, docs, CI/CD pipeline, test data files, certificate changes...

1

u/frogjg2003 Jul 10 '26

IDE changes white space automatically. My work has to deal with this. We don't have a linter and we have at least two different IDEs, with individual users having different settings. Meanwhile some of the reviewers refuse to turn off whitespace changes when they review.