2016-07-30 8 views
1

Ich bin ein Beitrag zu einem Repo, dessen Besitzer nicht sehr ansprechend ist. Ich möchte ein neues Feature entwickeln, aber zuerst muss ich einige Funktionen umgestalten (wird verwendet, um das neue Feature zu erstellen).GitHub Workflow der Eröffnung aufeinander folgenden PRs

Ich habe eine PR für das Refactoring geöffnet (immer noch darauf warten, zusammengeführt zu werden). Das Refactoring ist bereits groß genug für ein einzelnes PR. Jetzt möchte ich eine weitere PR für das neue Feature eröffnen. Ich könnte eine neue mit allen Commits von der vorherigen PR plus die Commits für die neue Funktion öffnen. Aber ich frage mich, gibt es einen besseren Weg, dies zu tun.

Oder sollte ich warten, bis der vorherige PR zusammengeführt wurde, bevor ich einen neuen öffne?

+0

Nicht wirklich relevant für StackOverflow. – Seiyria

+0

Wo soll ich dann danach fragen? – damonh

Antwort

1

Wenn die neue Funktion auf Ihrem Refactoring hängt, können Sie diese PR öffnen sofort, aber:

  • die neue Funktion Zweig von Ihrem aktuellen refectoring Zweig beginnen muss
  • die neue Funktion PR muss Folgendes enthalten in Der Kommentar, dass es darauf ankommt, dass PR xxx (das Refactoring PR) zuerst zusammengefügt wird
  • Wenn das Refactoring PR in irgendeiner Weise aktualisiert werden muss (neue Commits, Rebase, ...), müssen Sie Ihr neues Feature rebasen verzweigen Sie über den aktualisierten Refactoring-Zweig und forcieren Sie Push.
+0

Das ist im Grunde, wie ich das vorher gemacht habe, :) nur, dass ich in der PR stattdessen kommentiert habe, dem Besitzer zu sagen, dass er zuerst den anderen fusionieren soll. Nochmals, gibt es einen besseren Weg, dies zu tun? Oder genau das würden Sie tun, wenn Sie in meiner Position wären? – damonh

+0

@damonh Nein: GitHub unterstützt keine Abhängigkeiten zwischen PR, daher ist ein Kommentar im Moment am besten. – VonC

Verwandte Themen