Pull-Anfragen für ein 2-Personen-Team einführen - meine eigenen Anfragen zusammenführen?

10

Ich stelle einem Junior-Teammitglied (einem Koop) Git vor.

Sie sind jetzt mit den Grundlagen des Hinzufügens, Festschreibens, Drückens und Ziehens vertraut.

Jetzt möchte ich sie vorstellen, um Anfragen und Zweige zu ziehen.

Sollte ich das Gleiche für meine laufende Arbeit tun, wenn sie Pull-Anfragen in Filialen ausführen?
Ich werde derjenige sein, der ihre Pull-Anfragen zusammenführt. Ich war mir nicht sicher, ob es für mich am sinnvollsten wäre, in Filialen zu arbeiten (im Allgemeinen eine gute Praxis, die ich kenne, aber ich bin neugierig auf diese spezielle Situation von 2 Entwicklern mit einem Junior ) und wenn ja, bedeutet das, dass ich nur meine eigenen Zweige in Master zusammenführen werde. Würde ich sowieso eine Pull-Anfrage für meine Arbeit / Filialen machen? Im Allgemeinen verwenden wir für diese Änderungen den grundlegenden Github Feature Branch Workflow:
https://www.atlassian.com/git/tutorials/comparing-workflows/feature-branch-workflow

Gibt es einen Zweck für die Verwendung von Pull-Anfragen in meinem eigenen Repo, wenn ich der einzige Entwickler bin? ist nützlich, aber nicht ganz so spezifisch.

Der Arbeitsablauf mit 2 Personen an einem Projekt scheint ebenfalls allgemeiner zu sein

und

Sollte ich Pull-Anfragen von einer Filiale auf dem offiziellen Repo oder meiner Gabel öffnen? scheint mehr über Gabeln.

Michael Durrant
quelle

Antworten:

18

Nein. Sie sollten Ihre eigenen Pull-Anforderungen nicht zusammenführen. Was für die Gans gut ist, ist gut für den Betrachter. Das Zusammenführen Ihrer eigenen Pull-Anforderungen ist ein schlechter Präzedenzfall für unseren Junior-Entwickler. Es bedeutet auch, dass niemand sonst Ihren Code betrachtet. Egal wie alt wir auch sein mögen, wir alle machen Fehler und schreiben von Zeit zu Zeit schlechten Code. Bringen Sie Ihrem Junior bei, wie Codeüberprüfungen von der anderen Seite funktionieren, indem Sie ihn dazu bringen, Ihre Arbeit zu überprüfen und zusammenzuführen.

Er hat vielleicht nicht das gleiche Auge wie Sie, aber es wird ihn an den Prozess vom Ende des Rezensenten gewöhnen und er kann Sie überraschen und etwas Dummes fangen, das Sie getan haben. Minimal wird es Ihnen einen Hinweis auf Codeteile geben, die für Sie offensichtlich sind, die für ihn nicht offensichtlich sind. Dies hat einen zweifachen Vorteil.

  1. Sie beide lernen, wo Ihr Junior Lernaktivitäten konzentrieren muss.
  2. Sie lernen, wo Sie klüger sind, als Sie sein sollten.
Badeente
quelle
6
Der andere große Vorteil von Codeüberprüfungen besteht darin, dass mindestens zwei Personen jede einzelne Codeänderung gesehen, kennengelernt und die Möglichkeit hatten, Fragen zu stellen, bevor sie eingeht. Auch wenn der Junior-Entwickler nicht weiß, wonach er suchen soll, ist er es garantiert etwas daraus zu lernen.
Ixrec