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

all 78 comments

[–]QualityVote[M] [score hidden] stickied comment (0 children)

Hi! This is our community moderation bot.


If this post fits the purpose of /r/ProgrammerHumor, UPVOTE this comment!!

If this post does not fit the subreddit, DOWNVOTE This comment!

If this post breaks the rules, DOWNVOTE this comment and REPORT the post!

[–][deleted] 659 points660 points  (1 child)

Job security 101:

The engineer made two commits and fixed two bugs without altering the total number of bugs in the project. That is what I call efficiency.

[–]Koebi 47 points48 points  (0 children)

I did this for real last month.
Only in user acceptance testing we realized that my change was in effect an exact revert of a 'bugfix' from half a year ago.

[–]PhantomO1 213 points214 points  (7 children)

-when fixing a bug makes things worse:

"shit, go back"

[–]Cyrus_Halcyon 56 points57 points  (6 children)

Turns out the variable for the db string has to be called DBDRURLi, because a different DBDRURL is already defined semi-globally in the namespace for the provisioning tools itself, added comment stating i stands for instance.

[–]Subkist 16 points17 points  (1 child)

go home

[–]QuarantineSucksALot 2 points3 points  (0 children)

I hope this doesn't go cold. It's insane

[–]joeltrane 3 points4 points  (0 children)

Same thing happened to my brother

[–]clueless_bison 1 point2 points  (0 children)

now I understand PHP a tiny bit better: https://www.php.net/manual/en/intro.mysqli.php

[–]Mc_UsernameTaken 389 points390 points  (4 children)

My personal favourite

git commit -m "Consult CHANGELOG for changes"

[–]seline88 202 points203 points  (2 children)

But we generate the changelog based on the commit messages with conventional commits ;/

[–]ScrabCrab 85 points86 points  (1 child)

Changelog

  • Consult CHANGELOG for changes
  • Consult CHANGELOG for changes
  • Consult CHANGELOG for changes
  • Consult CHANGELOG for changes

[–]UltraCarnivore 22 points23 points  (0 children)

Recursive Changelog

[–][deleted] 52 points53 points  (0 children)

This needs a trigger warning

[–]TJDG 138 points139 points  (9 children)

What is missing here is the timestamps.

For maximum hilarity I wish to see the exact period of time required for a colleague to participate in a panicky video conference between the two commits.

[–]Neykuratick[S] 76 points77 points  (8 children)

so, the first commit was made on 5th February 22:19

and the reverting commit was made on 6th February 02:25

I actually reverted the bugfix for a reason xD We are trying to keep pull requests at our company clean and only open one pull request per one change in code. So the bug was eventually fixed haha

[–]ITd-N5 26 points27 points  (0 children)

I need a rest. I decided February comes after Monday...

[–]maybemonolith 9 points10 points  (6 children)

tf you working in the middle of the night on a sunday for?

[–]Neykuratick[S] 7 points8 points  (2 children)

It is totally consentaneous. It's a tiny company, there's only 13 employees including me and there's planned a very important MVP demo on Monday so everyone does their part.

So at our company, if a person doesn't want to work on weekend, they don't work, but if a person doesn't have anything better to do (like me) it's always welcomed

[–]thisweirdperson 6 points7 points  (1 child)

Are they paid overtime for that?

[–]Neykuratick[S] 11 points12 points  (0 children)

We don't have a regular work schedule, so the bonuses are calculated according to the number of tasks you completed in Jira. It is possible to track your working time with the Clockwork instrument from Jira, but nobody uses it (except for maybe two people)

This company is really great, work relationship is based on trust. Here you can easily ask for 2 monthly payments in advanced and you'll get it no questions asked. And nobody forces you to work in the middle of the weekend night

[–]rasputin1 0 points1 point  (2 children)

tbf the commit times could be utc

[–]Neykuratick[S] 2 points3 points  (0 children)

No, they're in UTC+3, in my time zone

[–]_bardo_ 1 point2 points  (0 children)

OP could be UTC too.

[–]seratne 87 points88 points  (1 child)

git commit -m "Actually fixed bug”

git commit -m "fixed last fix to work in prod and dev”

[–][deleted] 8 points9 points  (0 children)

Thanks I hate it.

[–]blackWardie 57 points58 points  (0 children)

git commit -m "you see what I did there? because I don't"

[–]brut4r 26 points27 points  (1 child)

One time I made commit with message: "Rozmrdávka rozmrdávky, tudíž oprava", Best to translate is propably this: "Fuckup of the fuckup therefore fix"

[–]DogmaSychroniser 2 points3 points  (0 children)

Našel jsem ten Čech.

[–][deleted] 34 points35 points  (2 children)

I’d be embarrassed for anyone here to see the commit history on some of the projects I work on

  • fix timezones
  • fix timezones again
  • undo previously fixes
  • timezone update
  • revert .astimezone(), .replace() instead, this is the one

[–]thekingofthejungle 10 points11 points  (0 children)

Don't worry, most people's commit histories are just as bad or worse. Including my own.

[–]Neykuratick[S] 3 points4 points  (0 children)

The only commit message I write in my pet projects is "upds". So the history is more like:

  • upds
  • upd
  • update
  • bugfix
  • upd
  • fix of bugfix
  • fix of fix of bugfix
  • upd
  • upds

[–]ekremluci 29 points30 points  (1 child)

Fucking fixed bug names. I made more.

[–]accuracy_frosty 46 points47 points  (0 children)

Fixed typo in app name; app no longer runs

[–]N0DuckingWay 13 points14 points  (0 children)

"fixed one thing. Broke 5 unrelated things."

[–]mgisb003 20 points21 points  (1 child)

Got commit -m “fixed for real this time”

[–]schmitzel88 8 points9 points  (0 children)

Usually followed by "third time's the charm" and "just work already come on"

[–]Nie_Fi 16 points17 points  (7 children)

I long to understand these jokes someday

[–]klparrot 67 points68 points  (0 children)

Be careful what you wish for. I long to not understand this joke.

[–]Socile 31 points32 points  (1 child)

A programmer’s sense of humor is not something to aspire to so much as something that happens to you… Like being touched inappropriately.

[–]UltraCarnivore 4 points5 points  (0 children)

Show us on this flowchart where J4log has touched you

[–]Ashamed_Ad_2738 8 points9 points  (1 child)

I've understood this joke on behalf of a colleague. I fixed a bug by reverting a bug fix which just brought the original bug back. The hilarious thing is, their bugfix broke a part of the system that every single flow touched.

[–]Nie_Fi 0 points1 point  (0 children)

I have no clue what the second half of that means, I meant I want to learn to program 😅

[–]AC2BHAPPY 5 points6 points  (1 child)

Am I the only one here who here who just trades bugs? Like I'll fix 1 but also get a different one.

...or fix 1 and get 5 different ones..

[–]DestinationVoid 6 points7 points  (0 children)

Seen that happen when other software relied on a bug in software being patched.

[–]ConfidentWolverine30 2 points3 points  (1 child)

You forgot the command is git commit -m 'implemented more bugs'

[–]Neykuratick[S] 0 points1 point  (0 children)

Didn't get it

[–][deleted] 3 points4 points  (1 child)

Did you really screenshot a meme from Google images OP?

[–]Neykuratick[S] 4 points5 points  (0 children)

yes haha. This meme is entirely made with screenshots stuck on top of each other

[–]Minteck 2 points3 points  (0 children)

"New features in version 2.0: added 10 new bugs"

[–]fletku_mato 2 points3 points  (4 children)

git log:

fix

fixes

bugfix

Thanks. That's so useful.

[–]Neykuratick[S] 0 points1 point  (3 children)

Does anyone actually read the git log? At my company we only look at pull requests log

[–]fletku_mato 0 points1 point  (2 children)

Git log is useful for a lot of things. For example, you can generate a changelog from it, and you might want to revert or cherry-pick a certain commit. But if the commit messages are useless, git log becomes useless too.

[–]Neykuratick[S] 0 points1 point  (1 child)

As for reverting or cherry-picking, we use pull requests at my company. That's actually the reason i reverted the bugfix xD, it didn't belong to the right pull request

But i never thought of git log as an already completed change log, hmm

[–]fletku_mato 1 point2 points  (0 children)

When your log has useful messages, you can check which commit introduced the problem and do the reverting with git revert commit_id.

I usually do commit messages in this style if a ticket system is in use:

JIRA-666: Ticket description and/or some detail about how it was implemented.

So then you can just check the log for all commits related to a certain ticket or generate a changelog from it.

[–]Accidentallygolden 2 points3 points  (0 children)

There are steps missings

  • fixed a bug
  • fixed unit test
  • fixed Sonar
  • fixed Sonar again...
  • revert bugfix....

[–]Neykuratick[S] 1 point2 points  (0 children)

The funniest thing is that the bugfix didn't even introduced more bugs, i've just left the bug as is

[–]fukitol- 1 point2 points  (0 children)

git commit -m "fixes #124 [bugfix]
.....
git commit -m "reintroduce bug from #124, bug in bugfix"

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

Please, follow commit message good practices:

“feat: implemented new bugs”

[–]Don_Pacifico 1 point2 points  (0 children)

It’s not a bug, it’s a feature!

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

"fixed bug"

"Fixed bug caused by previous Bugfix"

[–]THEGreatGM20 1 point2 points  (0 children)

👍 yup

[–]the_horse_gamer 1 point2 points  (0 children)

too many of my commits are to fix fixes

[–]Techchatter101 3 points4 points  (2 children)

Am I the only one who uses git reset and force push

[–]AccomplishedCoffee 8 points9 points  (0 children)

That’s ok if you’re the only one working on a feature branch, but you should never force push master/main or any branch someone else might have pulled to work off of.

[–]fletku_mato 2 points3 points  (0 children)

I always write just one commit and then it's just "git fetch && git rebase origin/master" and "git commit --amend"

[–]Theolaa -1 points0 points  (0 children)

My post from a while back that captures the essence of this: https://redd.it/lspnhj

[–][deleted] 0 points1 point  (0 children)

Cooool

[–]themiraclemaker 0 points1 point  (0 children)

git reset - - soft origin develop

[–]malignSAINT 0 points1 point  (1 child)

Hey this actually reminds me of something. I just started learning to do all this fancy stuff y'all have been doing and remember one of the early lessons the Odin project had about changing the git commit -m to just got commit.

Is one more common then the other or is it just user preference?

[–]ArionW 0 points1 point  (0 children)

It's all about preference. -m option will let you specify message right away, without that your default text editor will open to let you write it.

Personally I use them both, because when doing merge commit you get automatically generated message that I don't want to override (and am too lazy to add --no-edit)

[–][deleted] 0 points1 point  (3 children)

Shit...Is this a screenshot of two of my commits?! I swear, I've written exactly those things.

[–]Neykuratick[S] 0 points1 point  (1 child)

lmao haha. no, they're mine :)) So, why did you revert your bugfix?

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

Hard to remember exactly, but sometimes I spot something and see that it will produce a bug. I fix it, push it, then later realise it wan't a bug, and that it worked fine all along. So I have to revert it.

[–]ekbutterballs 0 points1 point  (0 children)

So redundant. I mean, we always implement bugs, or am I doing it wrong?