Meine Github-Pull-Anfrage wurde zusammengeführt. Wie sieht die Konvention derzeit aus?

17

Ich gabelte ein Projekt auf Github, nahm eine kleine Änderung vor und schickte eine Pull-Anfrage an den ursprünglichen Betreuer, der es eingezogen hat. Nun ist das letzte Commit da Merged pull request #11 from my_username/master.

Dies ist das erste Mal, dass ich dies tue, daher bin ich mir nicht sicher, wie die Etikette jetzt lautet: Ich habe ein git pull upstream masterund dann git push origin masterund jetzt das letzte Commit für mein eigenes Repository gelesen, Merged pull request #11 from my_username/masterwas sich für mich ziemlich seltsam anfühlt. Müssen die Leute das normalerweise so machen, muss ich irgendetwas tun, um "die Geschichte aufzuräumen" oder so?

Hinweis: Da es sich um eine winzige Dokumentationsänderung handelt, habe ich keine Zweige erstellt. Ich habe nur die Änderung in meinem masterZweig vorgenommen und die Pull-Anforderung gesendet. In diesem Teil muss also nicht aufgeräumt werden.

Sundar - Setzen Sie Monica wieder ein
quelle

Antworten:

10

Ja, das machen die Leute normalerweise, weil es das ist, was git macht.

Die Zusammenführungstests ergeben oft keinen Sinn. Dies ist das Ergebnis der "Schnellvorlauf" -Logik, dh das Zusammenführen erzeugt kein neues Festschreiben, wenn dies nicht erforderlich ist. Meistens wird beim letzten Commit für Master "Merged Master" angegeben, da der Entwickler Master mit seinem Zweig zusammengeführt und zum Zusammenführen gedrängt hat, was keine weiteren Zusammenführungen erforderlich machte. Es ist sowieso niemandem wichtig, Commits in git zusammenzuführen.

Jan Hudec
quelle
5
Wenn es Ihnen jedoch etwas ausmacht, können Sie die no-ffOption zum git mergeZusammenführen zum Master verwenden.
Xion