Programmer Humor

19703 readers

117 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

Keep content in english
No advertisements
Posts must be related to programming or programmer topics

founded 1 year ago

MODERATORS

977

Review Please (programming.dev)

submitted 8 months ago by JPDev to c/programmer_humor

112 comments fedilink hide all child comments

you are viewing a single comment's thread
view the rest of the comments

[–] [email protected] 5 points 7 months ago (3 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] 8 points 7 months ago (2 children)

you should meet my coworker. this is one week worth of work. and he still only commit once a week.

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

and he still only commit once a week.

WHYYYY?

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

Relatable

[–] shiftymccool 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

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

Many organizations vendor packages in the repo for a number of different reasons and languages. Not just for node.

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

Or maybe their IDE had a different auto indent config and they saved it all, then committed it all without checking the diff or the status.

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

You should have an agreed upon format that is enforced by cicd. Prettier, black, whatever.

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

I do like the idea of mandating git clang-format as the Kate project has.
That way the other devs don't need to change their own IDE settings to comply.