T O P

  • By -

jwaibel3

"Stupid #%!@& intern broke the repo again"


king_fart_123

who gave the intern the ability to do that?


42-monkeys

The ... uhm ... the office dog? \^\^'


Own_Possibility_8875

In this situation, it is not the intern who is stupid, it is the senior who doesn't know how to set up the branch protection rules


ninjaassassinmonkey

Or the management that is too cheap to buy GitHub enterprise...


Own_Possibility_8875

Aren't branch protection rules enabled even on free plan?


ninjaassassinmonkey

Not for private repos I think


Own_Possibility_8875

Hm my mistake then


Backlists

First one, imperative present tense. Good commit messages should be able to be preceded by the phrase “if applied, this commit will…” https://cbea.ms/git-commit/amp/#imperative


Big-Hearing8482

Great now I have to write “Break random shit no one saw coming”


42-monkeys

I'm not sure in what tense "small changes" is.


-Aquatically-

Small changed


rosuav

The plural of "small change" is "small changes". The past tense of "small change" is "oops I broke it again".


42-monkeys

And the future tense is "I'll rewrite this mess from scratch"?


fumanchumanfu

"Fiks" because if I use the word "fix" in any way in a commit with an AB# it automatically plops my ticket into closed and shoots over QA


AyrA_ch

Try `Fіx` (Note: This is not a standard `i`, hex of the text is `46 D1 96 78`)


Ved_s

default git merge message


Riverchimp_

This for sure. After doing the merge you don’t even need to type the message, just “git commit” and the default merge message is used.


anto2554

Iirc the jetbrains GUI requires a message


BirdlessFlight

Well there's your problem!


Grumbledwarfskin

Case ########: Fix merge conflict Fixed merge conflict.


stejzyy23

This is the way.


kaltschnittchen

Attempt to fix… Possible solution for…


seriousgourmetshit

I dno about you nerds but all my commit messages are ‘wip’


geek-49

If you're feeling tense about your commits, you haven't done enough testing :)


zoqfotpik

This one for sure fixes the merge conflict this time. Unlike the last 6 commits...


Mindless_Sock_9082

"Merges conflicted fixes"


vondpickle

Thus unto thee m'lord branch this merge conflicts hath been fixeth


PooSham

The correct one is not even there Fixes merge conflict


ongiwaph

Present third person is cursed


Phnx_212

I think that is implied by the right side.


Inappropriate_Piano

All of them are implicit in each other. The question is how you want to phrase it


Phnx_212

I don't think that's true. I interpret this post as how a PR can be responded to. You either comment telling original to fix the merge conflicts, or you fix them yourself.


Inappropriate_Piano

Your interpretation missed the part where the title of the post says it’s about commit messages. All of them are from the perspective of the person who thinks that they’ve fixed the problem (or equivalently, that the commit fixes the problem).


Phnx_212

You are right, I missed that, thank you!


Metworld

Red


Feisty_Ad_2744

Never celebrate early. It is not really fixed if it is not working in prod.


LegitimateMulberry

Fixing merge conflict


FantasticEmu

Team Give up, Delete local repo, try again


xXStarupXx

Merge conflict fix


MaskedBandit77

Hopeful, future tense. "This will fix merge conflict." Like I'm willing it into existence.


Unsey

WIP WIP WIP WIP WIP WIP


rosuav

This is the Indiana Jones of commit messages.


JackNotOLantern

Todo: fix <...> Commit message: fixed <...>


NUTTA_BUSTAH

Fix conflict: foo bar... of course