2014-04-11 6 views
13

Mein Team verwendet intern interne Pull-Requests zur Code-Überprüfung der von uns gepflegten Anwendung. Wenn es jedoch Zeit ist, ein Commit zu verschmelzen, drücken Sie einfach direkt auf master. Wir sind sowieso alle Repo-Collabos, und durch nicht mit der Pull-Anforderung, Code zu verschmelzen, vermeiden wir, unseren Commit-Verlauf mit Merge-Commits zu verschmutzen. Da die App in unserem Team intern ist, ist niemand anderes betroffen.Wie wird eine GitHub-Pull-Anforderung mit einer Commit-Nachricht geschlossen?

Was ich wissen will ist, kann ich meine Commit-Nachricht mit etwas wie "closes PR #30" ändern und habe Github automatisch die Pull-Anfrage für mich schließen?

Ich weiß, dass Sie Commit-Nachrichten verwenden können, um Probleme zu schließen, also hoffe ich, dass es etwas Ähnliches für Pull-Anfragen gibt. Ich habe online und SO schnell gesucht und nichts gesehen.

+1

[This] (https://help.github.com/articles/closing-issues-via-commit-messages) Hilfeseite von GitHub könnte Ihnen helfen. – ZeissS

+3

Danke @ZeissS, aber dieser Link adressiert nur das Schließen von * Problemen *, nicht das Ziehen von Anfragen. –

Antwort

10

Wahrscheinlich möchten Sie sich das GitHub help page ansehen. Dort heißt es:

Sie können eine der folgenden Schlüsselwörter verwenden, um ein Problem zu schließen über Commit-Nachricht:

  • schließen
  • schließt
  • fix
  • Fixes
  • geschlossen
  • behoben
  • Resol ve
  • löst
  • aufgelöst

So "Behebt # 123" oder "Resolved # 456" wird funktionieren. Alle Pull-Anforderungen werden als Probleme zugeordnet, sodass dies auch für Pull-Anforderungen funktioniert.

Hinweis: Sie sehen eine Meldung zu nicht zusammengeführten Commits, weil Sie die Pull-Anforderung geändert haben. Betrachtet man also die Pull-Anfrage, wird es nicht sofort offensichtlich, dass die PR eingebaut wurde (im Gegensatz zu einfach geschlossen), es sei denn, Sie haben etwas Bedeutungsvolles in die erste Zeile der Commit-Nachricht geschrieben, so dass Sie die Nachricht in der Pull-Anfrage sehen können.

+6

"Alle Pull-Anforderungen werden als Probleme zugeordnet, daher funktioniert dies auch bei Pull-Anforderungen." - Ahhh, * das ist der Teil, den ich vermisst habe! Wie kannst du das Wissen? Wenn github das als einen kurzen Satzzusatz zu der von Ihnen verlinkten Seite hinzufügen könnte, wäre meine Frage beantwortet worden! –

+1

Ich habe das getestet, und es hat funktioniert! Du hast Recht mit der Warnung "unmerged commits", aber mein PR sagt auch, dass "skingsland dies in vor 2 Minuten abgeschlossen hat." Jeder, der sich die PR ansieht und sich fragt, warum sie geschlossen wurde, könnte auf den Hash-Link zum Festschreiben klicken und sehen, dass sich das Festschreiben in "Master" befindet. –

+0

@SteveK Leider weiß ich das nur, weil ich GitHub schon eine Weile benutzt habe und weiß, dass Pull-Requests in der Problemliste erscheinen. Ich stimme dir zu, es wäre hilfreich, wenn GitHub es irgendwo prominent erwähnt hätte. :-) – jszakmeister