"Fix"
this post was submitted on 12 Dec 2023
808 points (99.0% liked)
Programmer Humor
20673 readers
2808 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 2 years ago
MODERATORS
With 400 lines changed over 50 files
"updates"
“feat: stuff”
Guilty of this one myself.
I had a commit recently that was like 2000 lines changed over 6 files. Really should have been a smaller issue.
Y tho??? Holy shit. Commits should be like functions. One thing and one thing only. Maybe a small group of files like the same change over multiple config files. 50 is insane to me.
'Change' if I'm feeling particularly chaotic.
git commit -m $(date)
See jira-blah: is my go-to. Sometimes there's even a jira at that location/number 🤔
load more comments
(1 replies)
Just use What The Commit.
You can also create a git alias:
git config --global alias.yolo '!git add -A && git commit -m "$(curl --silent --fail https://whatthecommit.com/index.txt)"'
Now you can just type 'git yolo' to create a commit!
"Make Sure You Are Square With Your God Before Trying To Merge This"
Full send.
Well such an informative reply! Thanks mate 👍
load more comments
(4 replies)
Psst,
git add -p
What does this?
"patch mode" - Patch mode allows you to stage parts of a changed file, instead of the entire file. This allows you to make concise, well-crafted commits that make for an easier to read history.
Highly recommend throwing --patch on any git commands you're used to using. You will have the prettiest, most atomic fkn commit, I'm serious people will love you for it.
I mean many people won't care, but the quality folk will notice and approve.
We make a singular commit per feature.
I always find this hard to follow personally.
load more comments
(3 replies)
Or just use a good IDE that makes doing atomic commits pretty natural.
load more comments
(4 replies)
Yay, learning!
Better yet, git commit -p
uuuuuuuu. and you could do -m to describe the commit.
next they'll add --push/-P.
perhaps add -r for fetch/rebase then commit.
one command to rule them all! 😈
git commit -m “changed somethings “
git push origin master
You forgot this --force flag.
I'm too lazy, I use -f
load more comments
(16 replies)
I’m using Copilot for it right now. It works on half of the cases.
That's about 300% better than my average!
For me, it was my boss gave me a programming task which he knew would take hours or a day or two... and then 15 minutes later tells me to "switch focus" and do a menial task that any of my five coworkers could do 🤦♂️
The usual reason would be "because coworkers"
That's in any bloody workplace! Especially if there is o synergy between different teams.
Forward three hours, me using thesaurus.com to try fit the whole gist of my change into the first line.
do git commit -v and then just summarize the diff you have in your editor in a human readable form.
Don't just summarize the content though, summarize the rationale or how things connect. I can read your diff myself to see what changed, I want to know the logical connections, the reason you did X and not Y, etc.
Or just say "stuff" and provide that context in the PR description separately, no need to overdo the commit log on a feature branch if you're using squash merges from your PR.
P1000x this.
I can read a diff.
I need to know why.
No, a code comment isn't good enough, it's out of date after the next commit.
load more comments
(3 replies)
Me trying to find ways around using the word "and" in the commit message.
git commit -m "directory_x:file_i.so: did x, y, and z; directory_x:file_ii.so: fixed t"
Oh god I feel so called out. I wish I paid more attention to my commit messages but I’m usually too busy fixing the directory structure and refactoring. Sigh.
load more comments
(4 replies)
You should not use -m, you should write commit body!
Why? My coworkers are barely literate and won't read anything with more than 4 or 5 words, writing a commit body would be a waste of time.
load more comments
(1 replies)
view more: next ›