JPDev@programming.dev to Programmer Humor@programming.dev · 8 months agoReview Pleaseprogramming.devimagemessage-square112fedilinkarrow-up1984arrow-down117
arrow-up1967arrow-down1imageReview Pleaseprogramming.devJPDev@programming.dev to Programmer Humor@programming.dev · 8 months agomessage-square112fedilink
minus-squareursakhiin@beehaw.orglinkfedilinkarrow-up5·8 months agoHuman 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.
minus-squareErwinLottemann@feddit.delinkfedilinkarrow-up8·8 months agoyou should meet my coworker. this is one week worth of work. and he still only commit once a week.
minus-squareuis@lemm.eelinkfedilinkarrow-up2·8 months ago and he still only commit once a week. WHYYYY?
minus-squareshiftymccool@programming.devlinkfedilinkarrow-up2·8 months agoThe only way I see that is a dependency update is if you’re versioning your node_modules or <insert-folder-here> which is generally a no-no
minus-squareursakhiin@beehaw.orglinkfedilinkarrow-up1·8 months agoMany organizations vendor packages in the repo for a number of different reasons and languages. Not just for node.
minus-squareulterno@lemmy.kde.sociallinkfedilinkEnglisharrow-up1·8 months agoOr 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.
minus-squarejjjalljs@ttrpg.networklinkfedilinkarrow-up3·8 months agoYou should have an agreed upon format that is enforced by cicd. Prettier, black, whatever.
minus-squareulterno@lemmy.kde.sociallinkfedilinkEnglisharrow-up0·8 months agoI 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.
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.
you should meet my coworker. this is one week worth of work. and he still only commit once a week.
WHYYYY?
Relatable
The only way I see that is a dependency update is if you’re versioning your node_modules or <insert-folder-here> which is generally a no-no
Many organizations vendor packages in the repo for a number of different reasons and languages. Not just for node.
Or maybe their IDE had a different auto indent config and they saved it all, then committed it all without checking the
diff
or thestatus
.You should have an agreed upon format that is enforced by cicd. Prettier, black, whatever.
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.