932
you are viewing a single comment's thread
view the rest of the comments
[-] gnutrino@programming.dev 19 points 4 months ago

This is why I always rename all the variables in the project on each PR.

[-] jol@discuss.tchncs.de 15 points 4 months ago

I know this is a joke, but it you did that I would reject the pr with the reason of too many things at once. Reopen separate PR to refactor variable names. I actually constaly get people doing this and it's dangerous exactly for the reason you're joking about. Makes it easier for errors to slip in.

[-] silasmariner@programming.dev 2 points 4 months ago

I know you're playing the straight man to a joke, but actually you can apply a linter, then tell GitHub to ignore the implied ownership history for the purposes of blame from that reclining pr. All such prs are massive and yet by virtue of the replayability of the linter it's also very easy to ensure errors didn't slip in when reviewing.

I know the original comment was about renaming all the variables, but that's obviously deliberately absurd, so I'm using here a completely realistic example instead.

[-] Lifter@discuss.tchncs.de 0 points 4 months ago

This will lead to change fatigue. People will rather not cleanup as they go anymore and just get the work done, with worse and worse code quality as a result.

[-] jol@discuss.tchncs.de 1 points 4 months ago

I prefer that than to sneak defects in huge PRs.

this post was submitted on 02 Jul 2024
932 points (98.5% liked)

Programmer Humor

19623 readers
1 users here now

Welcome to Programmer Humor!

This is a place where you can post jokes, memes, humor, etc. related to programming!

For sharing awful code theres also Programming Horror.

Rules

founded 1 year ago
MODERATORS