Lege commits naar remote schuiven

Ik heb één commit naar remote gepusht, maar nu realiseerde ik me dat het commit-bericht niet correct is. Ik zou het commit-bericht willen wijzigen, maar AFAIK is niet mogelijk. Dus besloot ik een lege commit te maken met het juiste bericht:

git commit --allow-empty

Zijn er nadelen/gevolgen van het pushen van lege commits? Is er een probleem waarmee ik in de toekomst te maken kan krijgen vanwege deze lege commit??


Antwoord 1, autoriteit 100%

$ git commit --allow-empty -m "Trigger Build"

Antwoord 2, autoriteit 32%

U zult geen verschrikkelijke gevolgen ondervinden, alleen de geschiedenis zal er wat verwarrend uitzien.

U zouhet vastleggingsbericht kunnen wijzigen door te doen

git commit --amend
git push --force-with-lease # (as opposed to --force, it doesn't overwrite others' work)

MAARdit zal de externe geschiedenis met die van jou overschrijven, wat betekent dat als iemand die repo in de tussentijd haalt, deze persoon erg boos op je zal zijn…

Doe het gewoon als u de enige persoon bent die toegang heeft tot de repo.


Antwoord 3, autoriteit 11%

het pushen van commits, of ze nu leeg zijn of niet, zorgt ervoor dat eventuele git hooks worden getriggerd. Dit kan niets doen of wereldschokkende gevolgen hebben.


Antwoord 4, autoriteit 6%

Zijn er nadelen/gevolgen van het pushen van lege commits?

Afgezien van de extreme verwarring die iemand kan krijgen over waarom er een heleboel commits zijn zonder inhoud op master, niet echt.

Je kuntde commit wijzigen die je naar remote hebt gepusht, maar de sha1 van de commit (eigenlijk het id-nummer) zal permanent veranderen, wat de source tree verandert — Je zou dan doe een git push -fterug naar remote.


Antwoord 5, autoriteit 6%

Zolang je duidelijk naar de andere commit verwijst vanuit de lege commit, zou het goed moeten zijn. Iets als:

Commit message errata for [commit sha1]
[new commit message]

Zoals anderen al hebben opgemerkt, heeft dit vaak de voorkeur om een gecorrigeerde commit te forceren.


Antwoord 6

Hoe zit het hiermee;

git commit --allow-empty-message -m ''

Other episodes