I mean, we've all done it, making some changes and the checking them in with messages such as "made some changes" or "fixed a bug." Messages so inane, so pointless, you might as well have written "magical fun bus" in their place (of this, I am guilty), as it would be, perhaps, more descriptive. I ask you then, what is the most pointless, most off topic, strangest, or just WORST commit message you have ever authored?
The worst must be no comment at all. And I am guilty... :-)
[insert friends name] is the bomb.
when you fix one of their errors. Became some sort of hype among friends after one started it when fixing useless stupid errors. Though the projects were not that big :).
Just checked git log
and I found Corrected mistakes (committed by me!) which I think is not a good commit message as it does not shed light on what mistakes have been corrected. I should have added a description.
Switched off unit test X because the build had to go out now and there was no time to fix it properly.
- Committed some changes
- Some bugs fixed
- Minor updates
- Added missing file in previous commit
I feel really sorry most of them were in school projects I promise :-(
My usual nondescript commit message:
oops
This usually results from committing more than was intended with svn commit
. However, now that I use Git, this essentially never happens. Not only do I not generally make the mistake of committing more than was intended (I don't use git commit -a
), but Git allows me to fix the bad commit before pushing the commit anywhere else!
"fixed errors in the previous commit"
Says it all really.. i.e. stuffed up one commit, then failed to even describe what was wrong (and what was corrected)..
Much worse than no message at all is the message that looks right but is wrong in subtle and misleading ways.
Can't take full credit for this one, but one of the web designers was tasked with adding corporate advertising to the internal homepage. Check-in message:
Added a banner to the default admin page. Please have mercy on me =(
'hmmmmmm'
Added that once when I couldn't even be bothered to think of something crap to write as a comment. very bad.
i see these:
bug fix
(the file has been there for 1 year and gone through 60 revisions... so "bug fix" doesn't mean anything).
typo
(when i check the diff, there actually is a bug fix. so i wonder, is "typo" just to make the mistake look not so bad).
[blank]
(the CEO and CTO like to use empty comment).
A cursory look at a project I've been doing the past one year where I did not use git seriously for the initial three quarters of the project year, revealed these among other commit messages:
- minor changes
- some brief changes
- assorted changes
- lots and lots of changes
- another big bag of changes
- lots of changes after a lot of time
- LOTS of changes. period
"Nothing to see here, move along" - When there were actually some major canges within the project. I just couldnt be bothered writing something useful.
“Obligatory placeholder commit message”.
Yeah, I wasn't lazy to type it out, but I was lazy enough to think of something better.
"Fixed the build"
While that does get to the point of why I'm commiting the change, it doesn't say anything about what I did change. That's why we have file diffs, right?
I feel this is just too good not to share. It was a programming assignment for our algorithms class. Apparently, we did most of the work after midnight; presented below are the more random/profane git commit messages:
- JOSH, WE WENT OVER THIS. EXPANDTAB.
- STEVE, WE WENT OVER THIS. C++ IO SUCKS.
- Some shit.
- add actual words
- I CAN HAZ COMMENTZ.
- giggle.
- Whatever.
- Finished fondling.
- FONDLED THE CODE
- this is how we generate our shit.
- unh
- don't be retarded.
- It works!
- unionfind is no longer being molested.
- Well, it's doing something.
- I'M PUSHING.
- Whee.
- Whee, good night.
- It'd be nice if type errors caused the compiler to issue a type error
- Fucking templates.
- I hate this fucking language.
- marks
- that coulda been bad
- hoo boy
- Fucking egotistical bastard. adds expandtab to vimrc
- if you're not using et, fuck off
- WHO THE FUCK CAME UP WITH MAKE?
- fix some fucking errors
- first blush
- uhhhhhh
- forgot we're not using a smart language
- include shit
- things occurred
- i dunno, maybe this works
- 8==========D
- whooooooooooooooooooooooooooo
- clarify further the brokenness of C++. why the fuck are we using C++?
the IT guy noticed revision 666 was coming up and wanted to make it special.
I convinced him that: "needs more cow bell" would be ideal.
a few months later the head of production was rather distraught because she thought the client had requested it.
One of my worst was quite recent:
I am even stupider than I thought
PROJ-1254: Some reasonable message
Turns out I made 4-5 commits with a completely wrong ticket id, it should have been PROJ-2154
Even worse: our issue tracking software looks up related code changes from the repository. Therefore, if somebody ever looks at that 6-month-old issue they will find a lot of random code commits that don't even make sense tied to it.
Oops.
Ready for it?.....
I did it for the lulz
!
We checked in an interface:
public interface ICanHasState
...this is in a respectable business package, no less. If it weren't for obfuscation, someone might actually find that.
The following are all real commits made by me on one project:
- A fix I believe, not like I tested or anything
- Useful text
- pgsql is being a pain
- pgsql is more strict, increase the hackiness up to 11
- c&p fail
- syntax
- ericflo sucks
- ERICFLO SUCKS
- fix
- someone fails and it isn't me
- Gross hack because eric doesn't know how to code
- totally more readable
- better grepping
- fix
- fix bug, for realz
- fix /sigh
- Does this work
- MOAR BIFURCATION
- bifurcation
- REALLY FUCKING FIXED
- FIX
- better ignores
- More ignore
- more ignores
- more ignores
- more ignores
- more ignores
- more ignores
- more ignored words
- more fixes
- really ignore ignored worsd
- fixes
- /sigh
- fix
- fail
- pointless limitation
- eppic fail eric
- ajax-loader hotness, oh yeah
- ericflo broke the regex, lame
- small is a real HTML tag, who knew.
- WTF is this.
- Do things better, faster, stronger
- Fixed a bug cause brosner said to
- Use a real JS construct, WTF knows why this works in chromium.
Back in 90's when we didn't have a revision control system, we were used to put change log entries at the top of the source code. Below are some of my entries from our GUI library. Jury shall consider that the defendant was 17 years old and a beginner in English back then :)
// 25th Sep 93 - 01:17 - I must sleep... it's working... in just three hours...
// 26th May 94 - 02:05 - SSG DID IT AGAIN!!!! THE ULTIMATE CHANGELINE
// DETECTION ROUTINES!!!!...
// 19th Aug 95 - 18:26 - I was wrong...
// 2nd Apr 94 - 21:12 - Completed with no bugs...
// 6th Aug 94 - 20:34 - Fixed a little bug...
// 10th Dec 94 - 02:11 - Fixed a bug in NoteLineCount... not seriously...
// 27th Aug 97 - 23:58 - woa!! this one was really HARD!
// 29th Jun 97 - 00:48 - Made it to compile...
// 13th Dec 94 - 01:04 - changed things...
// 26th Dec 94 - 00:22 - touched... (cs=2626)
// 31st Dec 94 - 00:21 - i think i fixed a bug... (cs=2605)
// 7th Oct 96 - 14:25 - perfect...
// 29th Nov 93 - 02:50 - Moved something to somewhere... goodnight...
Detailed description of an other change, which I made earlier to another file, simply because my IDE remembers the message I entered for that other commit. IMO this is much worse than a meaningless or empty message, since it's misleading.
"I'll explain this when I'm sober .. or revert it" .. thanks to git, I can't give a link.
got few times comments in foreign languages,,, good ones are the ones in which programmer speaks to the code ie. "got it bitch" ...
when i had improved performance of slow code,i changed engine started as ferrari engine started
oops
Looking back in the commit log, I did this 3 times in a row, about a month ago :-/
git commit -m "Inital Commit"
No description, and too often used... something more useful/creative?
here are some choice ones from our commit log:
- missing functions
- tightened up the java in my service call
- adding my
- story commit to release lock for joe
"good stuff only" .. but it's better in Swedish: "bara bra grejjor".
You know, like, it's good stuff, like. That's why I committed it, like. ;)
Classic for me and the other developer in the office used to be:
"Continued development..."
The worst I’ve personally done (a few times) was the following:
That’s not actually blank because a commit message was required, so I entered a non-breaking space (  / 0160).
I know, I know; it’s cheating. But to be fair, I’m the only one using this system, and the changes were really trivial.
"Does anyone read this? I'll be at the coffee shop accross the street."
Joel - 20/07/09
And the most prolific commit message I've used -
.
Besides the obligatory "Fixed some bugs" I had some "Final version" messages in my bachelor thesis svn repository logs. There were about 30 of them, evenly distributed among the last two days before the final deadline.
This isn't one I did, but one I saw:
D
Although this looks like a typo, what it really meant is "Converted source from using Guardian C system calls to using our proprietary D-series procedure wrappers."
(m)
Which means "minor change".
bugfix
then
Why has the +"*ç%&/ Visual Studio not added the file?
Although I have to admit that I don't regard any of these as bad: In the log, an "(m)" usually indicates: You don't need to look into this revision, the significant changes have been done before or after. The "Why has +"*ç...." indicates: This change has actually been explained in the last commit, so don't be surprised not to find it there.
No matter how often I asked, educated & pleaded, I used to work with someone who kept entering:
I change some files
She left before I killed her.
On revision 2013 of my current project:
We made it past 2012 and the world did not end!
Sometimes I run these for shits and giggles:
git log --oneline | grep WTF
git log --oneline | grep thingy
git log --oneline | grep oops
It's usually good for a laugh.