this post was submitted on 29 Mar 2024
977 points (98.3% liked)

Programmer Humor

19623 readers
26 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
 
(page 2) 50 comments
sorted by: hot top controversial new old
[–] [email protected] 5 points 7 months ago (2 children)
[–] [email protected] 5 points 7 months ago (7 children)

Human made changes is likely not what caused this image to occur.

111 files with that kind of change count is most likely a dependency update. But could also be that somebody screwed up a merge step somewhere.

[–] [email protected] 2 points 7 months ago (1 children)

The only way I see that is a dependency update is if you're versioning your node_modules or which is generally a no-no

load more comments (1 replies)
load more comments (6 replies)
[–] [email protected] 2 points 7 months ago

I'm still annoyed that Github doesn't have good support for stacked diffs. It's still not possible to say that one PR depends on a different one, and still has no ability to review and land them as a stack.

[–] [email protected] 4 points 7 months ago* (last edited 7 months ago) (1 children)

Love it when my coworkers reformat the code style, making it nigh impossible to understand what they actually changed, while greatly inflating their “contribution.”

It also blows away the git blame, making it hard to know who actually changed that one critical line of business logic 3 years ago that you need to understand before trying to fix some obscure bug.

I have one coworker who does this constantly and if you just looked at git blame, you’d think he wrote the entire code base himself.

[–] [email protected] 2 points 7 months ago

First things first: Your team needs a coding style.

Also: With git reflog ignore-revs you can filter commits that only adapt the style.

And while we're at it, check out the -C -C -C flag for git blame. https://git-scm.com/docs/git-blame#Documentation/git-blame.txt--Cltnumgt

[–] [email protected] 2 points 8 months ago

Just one review request?

load more comments
view more: ‹ prev next ›