Programmer Humor

19564 readers

643 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

[email protected]

977

Review Please (programming.dev)

submitted 7 months ago by [email protected] to c/[email protected]

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 (10 children)

Keep changes small, we use git patch stack https://github.com/uptech/git-ps

[–] [email protected] 5 points 7 months ago (8 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] 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.

load more comments (6 replies)

load more comments (7 replies)