2016-04-11 2 views
0

Angenommen, Sie überprüfen eine Pull-Anforderung und Sie sehen eine Möglichkeit, Code zu verbessern. Empfehlen Sie dem Entwickler, das ursprüngliche Commit zu ändern oder ein neues Commit mit der Änderung und einer Commit-Nachricht zu erstellen, die erklärt, warum es geändert wurde?Soll man Commits in einem Feature-Release ändern, anstatt ein neues Commit hinzuzufügen, das einen Fehler korrigiert?

+0

es ist wirklich eine Frage der Meinung. Manche Leute bevorzugen, dass jede PR nur 1 Commit hat, manche bevorzugen mehrere Commits ... – pedrorijo91

Antwort

2

Persönlich würde ich empfehlen, ein neues Commit zu erstellen.
Auf diese Weise wäre der alte Code immer noch verfolgbar, falls sich später herausstellen sollte, dass das Gegenteil der Fall ist.
Wenn es mit allen Commits unordentlich wird, können Sie immer neu erstellen :)

Verwandte Themen