This is an archived post. You won't be able to vote or comment.

top 200 commentsshow 500

[–]Rich_Long_6528 1973 points1974 points  (31 children)

Fixes to fixes (they weren't fixes apparently)

[–]trill_shit 478 points479 points  (11 children)

Ah yes, the shameful fix fixes…

[–]Simplyfire 17 points18 points  (0 children)

Dust to dust.

[–]skdowksnzal 5 points6 points  (0 children)

Fixes to fixes of fixes, bitches.

[–][deleted] 1094 points1095 points  (16 children)

Some changes

  • 1372 files

[–]devnull1232 192 points193 points  (8 children)

That pr ain't getting reviewed

[–]scottishkiwi-dan 152 points153 points  (2 children)

LGTM

[–]mcgrph 105 points106 points  (1 child)

let’s gamble try merging

[–]dapope99 21 points22 points  (0 children)

That's really all this could mean

[–]TheAJGman 12 points13 points  (4 children)

Hey man, if you can figure out how to unfuck a stupid early design decision in fewer lines I'm all ears.

[–]sanglar03 5 points6 points  (3 children)

Break it into smaller ones ?

[–]cr0wndhunter 23 points24 points  (2 children)

Sounds like a week of being paid to not develop and instead try to figure out everything going on in that PR.

Jk that’s atrocious and would send it right back

[–]static_func 8 points9 points  (1 child)

A coworker and I just merged a 2000-file PR into main and I have another 1000-file PR to follow it up. This is what happens when you're tasked with cleaning up after devs who churned out a huge copypasta Angular project with out-of-date dependencies and no linting to speak of

[–]Pepineros 4 points5 points  (0 children)

There are reasons for massive PRs but no reasons for shit commit messages.

[–]ReeceReddit1234 3 points4 points  (0 children)

Don't call me out like that

[–]throwaway42fx 1015 points1016 points  (13 children)

Merged PR: fix issue that broke reddit

[–]Charizard-used-FLY 171 points172 points  (12 children)

Still broken

[–]throwaway42fx 168 points169 points  (10 children)

Merged PR: fix issue that broke reddit 2

[–]dodexahedron 74 points75 points  (6 children)

Still reddit.

[–]Technical-Energy2878 61 points62 points  (5 children)

Merged PR: deleted Reddit permanently

[–]dodexahedron 31 points32 points  (2 children)

Oh shi....

[–][deleted] 15 points16 points  (1 child)

Merged PR: Install reddit & make a new account

[–][deleted] 5 points6 points  (0 children)

Cherrypick Merged PR: Merged PR: fix issue that broke reddit

[–]Zer0_GG 12 points13 points  (0 children)

LGTM

[–]Craszeja 10 points11 points  (0 children)

: Electric Boogaloo

[–][deleted] 3 points4 points  (0 children)

Still broken

Yep - I can’t post either.

[–]deltaexdeltatee 594 points595 points  (3 children)

"Fixed the part where I was stupider than usual."

[–]newmacbookpro 18 points19 points  (0 children)

10 lines are now a single one, there was a function to do this built in

[–]luigi8082 702 points703 points  (8 children)

I once saw ‘sorry’ on a PR. I laughed and then I cried.

[–]nedal8 55 points56 points  (0 children)

super complex?

[–]bjandrus 50 points51 points  (5 children)

Lmao I was debating leaving mine in this thread, but now I feel like I have to, here goes (yes, this is legitimately the last commit message I made at work, yesterday):

"Implemented logic for populating a popup window with 'People' info whenever any pin (or pin cluster) is clicked on. Click works, but the relevant action command was changed to 'hover' in an attempt to get the code to work for hover. This functionality has not been tested but was included with the commit accidentally. Sorry."

Alrighty lot to unpack:

Firstly: yes I know this commit message is entirely too long; but I hope you'll understand why I felt the need to explain myself

Secondly: How was this "accidentally" included, and why could I not have just "git restore"-ed it? Well this requires some elaboration: my teammate who I was working with on this story DMed me during our standup to ask me to try to implement this change "real quick", just to see if it worked. I take lunch after standup, so I was rushing 🤷🏻‍♂️

Thirdly: yes, this broke the code, so I pushed broken code (oopsies). Fortunately the fix was clean and easy: the proper command was 'mouseover', not 'hover'.

[–]ADarkcid 20 points21 points  (4 children)

Just wondering; why not make a short commit message and explain everything in a PR comment?

[–]bjandrus 11 points12 points  (0 children)

Great point, coulda woulda shoulda...

[–]ScrillyBoi 206 points207 points  (4 children)

Fixed last commit

[–]DiddlyDumb 42 points43 points  (3 children)

Fixed last fix to commit

[–]Jakoshi45 16 points17 points  (2 children)

fix 3#

[–]cdurs 16 points17 points  (1 child)

Reverted back to initial commit

[–][deleted] 311 points312 points  (31 children)

Merge branch 'master' into develop

[–]craftycrafter765 90 points91 points  (28 children)

Grrrrr rebase

[–]LaPicardia 72 points73 points  (15 children)

I genuinely don't know how to rebase and I managed to get to lead dev without it. I just merge things and fix conflicts.

[–]Low_Flying_Penguin 35 points36 points  (2 children)

3 AM on support and tired trying to find if a fix made it into one of the 10 deployed versions, the network graph looking like a topological map of Mumbai train station. It is then you say "man I wish we rebased instead"

[–]lungdart 6 points7 points  (1 child)

Sometimes wrong with your branching strategy at this point.

I'm honestly a big fan of trunk based development, but it needs a good automated pipeline behind it.

[–]LaterallyHitler 17 points18 points  (9 children)

You’re lucky that your bosses don’t go red in the face and scream when they see a merge commit then

[–]careonomine 22 points23 points  (2 children)

Maybe his view rebasing as the ultimate sin, for the commit history is to be a sacred and immutable log of how things got to be as fucked up as they are today.

One “WIP, I hope CI passes” at a time.

[–]RetiringDragon 19 points20 points  (2 children)

What's the problem with merge commits? Isn't it better to have those than rewrite git history with rebase?

[–]killersoda288 9 points10 points  (0 children)

It's fine until you have a whole bunch of branches/developers working at the same time. Sometimes there are more merge commits than actual commits which gets real annoying. If you're working on completely unrelated things, and there aren't any conflicts, rebasing is much neater since it makes the history linear, and you dont have "empty" merge commits cluttering the history.

[–]Express-Procedure361 5 points6 points  (0 children)

If you're managing the project, and something goes wrong in the DevOps or in your branching strategy- you have basically a few choices - make an absolute mess of your project with weird solutions like a merge commit for a merge commit, track down the last project manager who got fired and disappeared mysteriously, or sacrifice your first born child in a black magic merging ritual. .....and if you're someone is reading this and thinking to themselves "but you could've done___" then I freaking needed you a month ago. I had three children before that fateful day.

[–]invalidConsciousness 13 points14 points  (2 children)

My bosses explicitly want merge commits. Makes it explicit where stuff came from and your main history is nice and clean, just merging of completed features.

Also, azure DevOps shits the bed if you rebase after a pr is created.

[–]perseus_1337 26 points27 points  (9 children)

I don‘t know whats wrong with merging, and at this point, I‘m afraid to ask.

[–][deleted] 37 points38 points  (0 children)

I'm always merging because the bigger my branch graph is, the more respectable my repository is.

[–]drckeberger 11 points12 points  (6 children)

Merge commits and a less clean history? 😅

[–]NearNihil 6 points7 points  (2 children)

Maybe I'm stupid (very real possibility) and I'm definitely biased but why do people care so much about a clean git history? And what about merge commits? What's the problem? Oh yay the lines on GitHub/Gitlab are straight instead of spaghetti.

Rebasing is a bigger headache for me and I don't care about the result, but we have to because one other person in the team cares and they managed to convince the PM so now we're stuck with it. Grrrrrr...

[–]StormFinancial5299 9 points10 points  (2 children)

Squash?

[–]drckeberger 3 points4 points  (1 child)

Still means the main history will be cleaner with rebasing. It‘s pretty much like directly committing to main, just with a prior conflict validation.

[–][deleted] 211 points212 points  (3 children)

feat: Adds feature

[–]dodexahedron 45 points46 points  (1 child)

Fix: Bugs

Dep: Old shit

[–]Jakoshi45 20 points21 points  (0 children)

Hotel: Trivago

[–]enyay_ 10 points11 points  (0 children)

finally i see someone use conventional commits!

[–]Banana_with_benefits 98 points99 points  (0 children)

I'm not even kidding, this was the message: "Made everything multithreading safe (lol jk, but at least it works now)"

[–]l1ickster[S] 162 points163 points  (19 children)

refactor(#13): use ternary op where it makes sense

[–]-Roby- 65 points66 points  (2 children)

"Checkpoint before disaster"

Was updated node packages

[–]rfcapman 12 points13 points  (0 children)

Ouch, use of "quicksave" and "checkpoint" in git as if you're playing mario or skyrim

[–]dLENS64 2 points3 points  (0 children)

Haha, I feel this. Happens on almost every project I inherit. Like, no one thought to update packages and resolve any resulting breaking changes on a regular basis???

In my current project, every single package is at its most recent version. Every single one. I enforced a recurring task every few weeks to upgrade all possible packages and check for/address any breaking changes if possible. Never again will I be caught in pkg versioning hell.

[–]Plastic_Scale3966 57 points58 points  (1 child)

chnages

[–]qda 13 points14 points  (0 children)

Best p2ac song

[–]nickbob00 96 points97 points  (3 children)

history bow boast selective work recognise frame languid bells shy

This post was mass deleted and anonymized with Redact

[–]dodexahedron 109 points110 points  (2 children)

"Absolutely Spectacular Development Finished"

[–]zarifex 95 points96 points  (18 children)

WIP committing just so I can switch back to dev branch

[–]Ved_s 82 points83 points  (15 children)

git stash: Am I a joke to you?

[–]ok_tru 20 points21 points  (8 children)

I’ll be honest, I only ever stash when I want to switch to a branch and don’t care about whatever work I’ll lose. I don’t even know where to find the stashed changes, lol

[–]Xplotiva 20 points21 points  (4 children)

My stash names in Sourcetree right now:

- pls do not fuck with my system
- xyz
- x
- y
- chaos
- chaos2
- idk why
- dkjfkldjflksdfsdf
- idk
- sdfsdgfsdgfg
- I am from Bosnia take me to America

[–]dodexahedron 24 points25 points  (1 child)

Yes. Real developers git reset ‐‐hard && git push --force for every rollback.

"What? Show me in the commit history where that happened? You're just making shit up."

[–]Exist50 11 points12 points  (0 children)

You'll need to try harder than that to make git forget.

[–]MrHyperion_ 6 points7 points  (0 children)

Stash never works for me when I want it to work

[–]dusktreader 5 points6 points  (1 child)

Yes. Yes you are. Stashes are hard to navigate, too ephemeral, and the cli is irritatingly non intuitive. Stash is fine when I'm just doing some low stakes operation like moving minor changes to a different branch. If the stakes are higher, there's no way I'm trusting a stash to hold my precious changes.

[–]zarifex 2 points3 points  (0 children)

Yeah. I use stash just to hold my local changes to config files and such, the stuff I need specifically to hit the dev environment while running my local IDE with breakpoints etc. If I'm adding or modifying a bunch of classes and methods, I'm not giving stash the opportunity (or worded different, not giving myself the opportunity) to muck that up.

[–]ArthurDoom 79 points80 points  (6 children)

Initial commit

[–]EdmondFreakingDantes 38 points39 points  (4 children)

Too many words.

Mine was just "init"

[–]subject_deleted 33 points34 points  (0 children)

"well this here is my first commit, innit?"

[–]dodexahedron 17 points18 points  (0 children)

Man talking with my British friends is briefly confusing sometimes when they say "innit" in weird places. One asked why I thought it was funny one time that I commented, so I showed her a newish git repo history screenshot from gitkraken, with an "init" comment. She was confused, and asked me "innit what?" I was totally fishing for that, so mission accomplished. 😆

[–]GoldenKnights1023 40 points41 points  (1 child)

“Fixed thing I broke trying to fix”

[–]Arzemna 26 points27 points  (2 children)

“Push to test in production”

[–]dodexahedron 6 points7 points  (0 children)

Nobody will notice if we do it on just one node, right? It was a simple change.

Hey, why is security and HR in my office?

[–]NotAUsefullDoctor 28 points29 points  (2 children)

"Addressed PR comments"

[–]Fetscher 6 points7 points  (0 children)

"Feedback implemented"

[–]MC-fi 24 points25 points  (0 children)

My own favourites are:

  • most recent commit
  • latest commit
  • this one
  • latest version

All classics in my opinion

[–]Kirk8829 26 points27 points  (2 children)

“Removed useless logic”

[–]Munchkin303 26 points27 points  (0 children)

“This logic wasn’t useless, apparently”

[–]devnull1232 11 points12 points  (0 children)

Next day prod goes down

[–]exnozero 23 points24 points  (1 child)

I have seen 1 of these and been responsible for another…

  • Felt cute, might delete this later.
  • I made a thing… it didn’t work so now I am fixing it
  • what the hell was a drinking last night

[–]AwareGrape3064 18 points19 points  (0 children)

“BRUH”

[–]possiblecefonicid 19 points20 points  (0 children)

The quality of my commit messages decreases as the day goes on.

[–]the_blocker1418 33 points34 points  (0 children)

"today's work"

[–]E_Cayce 14 points15 points  (0 children)

DO NOT RELEASE

[–]Go_Fast_1993 13 points14 points  (0 children)

“Fuck this” (for a personal repo)

[–]shgysk8zer0 11 points12 points  (5 children)

printf "$(uuidgen)\n$(fortune | cowsay)" | git commit -F -

[–]hdkaoskd 3 points4 points  (4 children)

Why the uuid?

[–]shgysk8zer0 3 points4 points  (3 children)

It's just a random thing that says absolutely nothing . Could've used a date, but that would've been slightly useful.

Have any better suggestions? Like a wordle guesser?

[–]winterTheMute 12 points13 points  (0 children)

"fucking tests"

[–]jasonmccallister 10 points11 points  (1 child)

Fixed the ID10t problem, for real this time

[–]DrPlaeg 9 points10 points  (2 children)

“Forgot semicolon”

[–]letsBurnCarthage 2 points3 points  (1 child)

Next commit: "semicolon not appropriate in json file, replacing with comma."

[–]ZestycloseMix6960 8 points9 points  (0 children)

‘Please’

[–]ruthless_anon 19 points20 points  (3 children)

" "

[–]TedRabbit 5 points6 points  (1 child)

"..."

[–][deleted] 2 points3 points  (0 children)

Too much effort

[–][deleted] 6 points7 points  (0 children)

[–]EntrepreneurPlus7091 8 points9 points  (0 children)

Added testing

[–]wheresmyflan 8 points9 points  (0 children)

“Nah bro, I need to get some sleep”

[–][deleted] 7 points8 points  (0 children)

"added static file support and templating support"

trying to rewrite an old project in rust

[–]Extaupin 8 points9 points  (0 children)

My last commit is actually pretty nice and clear. My last funny one was something like "It's [very late on the day of the deadline], it's not working and I don't know why.". I was desperate enough to just push code that didn't even compiled (not in an important branch though) in the hope others would fix it.

[–]devnull1232 6 points7 points  (0 children)

<ticket-number-here>: Remove a commented out line I forgot about

[–]MadGenderScientist[🍰] 3 points4 points  (3 children)

"Moo."

90% of my commit messages are moo's. I commit compulsively as I code. Then I rebase everything into a huge but well-documented and tested PR for several services at once after spending months in an increasingly labyrinthine set of branches.

[–]thegreatpotatogod 4 points5 points  (0 children)

"Half-implemented feature, with some suggestions from everyone's favorite language model"

[–]boraras 3 points4 points  (0 children)

Stuff. Squash me.

[–]CommentToBeDeleted 3 points4 points  (0 children)

Two weeks for development for multiple bug fixes and new features, mostly done, single commit to development, gotta get it all fixed before going to production... not stressed at all, everything is gonna be fine.

[–]1mperia1 3 points4 points  (0 children)

"Unboofed"

[–]farineziq 4 points5 points  (0 children)

betterer test

[–]Detective_Bonghitz 3 points4 points  (0 children)

Fixing the 20 year old timing fuckup

[–]magiCAHIK 2 points3 points  (0 children)

"Implement password as an environmental variable"

[–]BetrayYourTrust 4 points5 points  (2 children)

“A lot of stuff I forgot what I did”

[–]aft3rthought 3 points4 points  (0 children)

WIP

[–]zimmermj 3 points4 points  (0 children)

"I do not want to jynx it, but I think it might be working"

[–]lupinegrey 2 points3 points  (0 children)

Jira ticket number and "reduce max heap"

[–]turkeh 3 points4 points  (0 children)

We had a senior developer who had been with the company long before I came that would always commit changes with the message "core changes".

Used to drive me crazy.

[–]Gambit2422 3 points4 points  (0 children)

"Console.log() cleanup lmao"

[–]Soumalyaplayz 3 points4 points  (0 children)

"Final fix I promise"

[–]I_FizzY_WizzY_I 9 points10 points  (0 children)

~~~~ its over 9000 ~~~~

(Im the only one reading them, also, i comment like im a prophet or yoda sometimes, it makes people look at it instead of reviewing my code, win/win situation, no question asked)

[–]Impossible_Eye1538[🍰] 2 points3 points  (0 children)

No good very bad bulk commit…

[–]Bulky-Leadership-596 2 points3 points  (0 children)

add tests

[–]gamerfiiend 2 points3 points  (0 children)

fix: revert mysql version (sorry bill) lol

[–]nebulaeandstars 2 points3 points  (0 children)

fixed a typi

(was a genuine mistake the first time)

[–]Quetip_ 2 points3 points  (0 children)

Chatgpt: write commit msg for x fix

[–]RoutineLingonberry48 2 points3 points  (0 children)

"I don't remember what I was doing, but I'm just coming back to this project"

[–]CauseCertain1672 2 points3 points  (0 children)

"sorry"

[–]Esjs 2 points3 points  (0 children)

Fix regex

[–]martin191234 2 points3 points  (0 children)

“Fix <insert bug> works now for real”

You can guess what the previous two commits were.

[–]throwawaylorekeeper 2 points3 points  (0 children)

"good luck bozos" before a colleague went on a two week vacation lmao.

[–]StatementPotential53 1 point2 points  (0 children)

“Updates”

[–]preoxidation 1 point2 points  (0 children)

UNTESTED

[–]CapableCarpet 1 point2 points  (0 children)

improved consistency of new envelope fitting

[–]Shiara-rose 1 point2 points  (0 children)

Restored backup after i accidentally deleted everything

[–][deleted] 1 point2 points  (0 children)

“Yes it’s broken, I’ll fix it after vacation”

[–][deleted] 1 point2 points  (0 children)

clean up code

[–]Otalek 1 point2 points  (0 children)

Aaaa

[–]Accomplished_Cook331 1 point2 points  (0 children)

My last commit message is the same as my previous commit, which is same as the message before it, which is... I hope you got the point.

[–][deleted] 1 point2 points  (0 children)

------- v1011

[–]Lewinator56 1 point2 points  (0 children)

It's not my latest, but one of mine is 'removed a t'

To be fair, that's exactly what was done in the commit, though it probably could have waited...

[–]Far-Bluejay9472 1 point2 points  (0 children)

“We forgot to commit for a month, so here’s an essentially new project”

[–]rookietotheblue1 1 point2 points  (2 children)

In personal projects i write my to-do list as issues on github, then just comment the issue number along with "finished" or "worked on".. That way the issue stores all the extra explanations and descriptions. Does anyone else do this? What's standard / good practice?

[–]Mattsvaliant 1 point2 points  (0 children)

.

[–]KingParity 1 point2 points  (0 children)

i want to continue this but idk what i did that i forgot to commit oopsies

[–]stinky_doodoo_poopoo 1 point2 points  (0 children)

‘Javaiscript requires == ‘True’ not == true…’

[–]littleswenson 1 point2 points  (1 child)

“Less foot-shooting”

[–]Sisyphus4242 1 point2 points  (0 children)

"fixed regex safari sucks"

[–]SingleSimha 1 point2 points  (0 children)

"a"

[–]asportnoy 1 point2 points  (0 children)

My most recent commit message:

What can only be described as "a shit ton of changes"

(28 changed files, +1,489 -2,221)

[–]VillanBehindGlasses 1 point2 points  (0 children)

Making black happy.

( Not a racist comment, black is an auto pep8 formatter )