-
Notifications
You must be signed in to change notification settings - Fork 0
pull requests are bad
Andre Winkler edited this page Nov 19, 2023
·
3 revisions
Link: https://www.youtube.com/watch?v=ASOSEiJCyEM&t=373s
-
Pull Requests nicht approven, wenn offensichtliche Fehler vorliegen
-
Kommentare beantworten. Ggf. begründen / oder aus Respekt zumindest antworten...
-
Vorschläge allein berechtigen nicht für das Ablehnen von Pull-Requests.
-
Vertrauen in die Teammitglieder ist die Grundlage für eine gute Zusammenarbeit.
-
D.h. ich vertraue meinen Kollegen, dass diese ihre Änderungen nach besten Möglichkeiten umgesetzt haben.
-
So wenig Code wie möglich, um eine Anforderung umzusetzen. Aber nicht so wenig Code, dass es keiner mehr versteht. KISS halt... Und selbst hier gehen die Meinungen teilweise weit auseinander.
by Andre Winkler 2021, Hamburg