974
Review Please (programming.dev)
you are viewing a single comment's thread
view the rest of the comments
[-] ursakhiin@beehaw.org 5 points 7 months ago

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.

[-] ErwinLottemann@feddit.de 8 points 7 months ago

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

[-] uis@lemm.ee 2 points 7 months ago

and he still only commit once a week.

WHYYYY?

[-] ulterno@lemmy.kde.social 0 points 7 months ago
[-] shiftymccool@programming.dev 2 points 7 months ago

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

[-] ursakhiin@beehaw.org 1 points 7 months ago

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

[-] ulterno@lemmy.kde.social 1 points 7 months ago

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.

[-] jjjalljs@ttrpg.network 3 points 7 months ago

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

[-] ulterno@lemmy.kde.social 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.

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

Programmer Humor

19594 readers
764 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