Versionskontrolle / Pull-Request ablehnen?

[M3Y3R]

Hallo zusammen,

ich habe mal eine Frage, die ich bisher so leider nicht lösen konnte.

Wir verwenden zur Versionskontrolle "Bitbucket".
Nun ist es so, dass neue Features oder auch Bugfixes in einer separaten Branch durchgeführt werden.
Im Anschluss daran wird dann ein Pull-Request erstellt, welcher auf den "Master" zeigt.

Wie geht ihr mit solchen Pull-Requests um?
Wie werden die Prozesse bei euch geführt und gelebt?
Was passiert, wenn bei der Kontrolle weitere Probleme auffallen?
Kommentiert ihr die betroffene Stelle nur oder lehnt ihr den gesamten Pull-Request ab?

Generell würde mich mal interessieren, wann ein Pull-Request abgelehnt wird.

Über regen Austausch freue ich mich natürlich ;-)


Viele Grüße
M3Y3R

 

[theHacker]

Ich denke bei diesem Thema an den GitHub-Workflow, die einen Pull-Request nicht nur zum "Nimm meinen Code ab, ich habe fertig" benutzen, sondern auch zum "Hey, ich hab n Problem. Guck meinen Code an und schreib was dazu" mitten in der Entwicklung nutzen.

Konkret mit "Bitbucket" kenn ich mich jetzt nicht aus, aber da es auf Git basiert, geh ich doch davon aus, dass ein abgelehnter Pull-Request nicht gleich das Ende des Branches is. Also entweder

• annehmen und mergen
• weiter kommentieren/diskutieren, schlussendlich ablehnen und weiterarbeiten; und nach einiger Zeit eben einen neuen Pull-Request auf denselben Branch machen