this post was submitted on 04 May 2024
322 points (92.8% liked)
Programmer Humor
19488 readers
1147 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
you are viewing a single comment's thread
view the rest of the comments
view the rest of the comments
Maybe. I do more DevOps these days, so tend to have many small changes that can’t even be tested without checking them in and running in CI. I’d have hundreds of “fix unit tests” commits alone
We squash. I'm not really interesting in your local journey to land the change. It's sometimes useful during review, but after that it's mostly the state of the main branch I care about. It's what I need to bisect anyway.
I don't like commits that are just references to issues. Copy the issue into the commit message so
git blame
tells you something useful. Unless it's just closing a simple big. Then the title and issue reference are plenty.Depends on the project I imagine.
I've been thinking about that when talking to my DevOps colleagues, that there's gotta be a better way to test CI before committing. The whole change-commit-test dance would kill me if on a daily basis. So cumbersome.