On 10/15/17 17:35, Johannes Rudolph via Dev wrote:
Am 15.10.2017 um 15:29 schrieb Jan-Tarek Butt via Dev dev@lists.ffnw.de:
Wir arbeiten über MRs
Also wenn ich mir die Historie so anschauen im Site.conf repo dann sieht das aber ganz und gar nicht da nach aus!
Der MR für v2017.1.x war über ein Monat offen.
Und wenn du mal einen Merge Request erstellt hast hast du den selber gemerged...
Ja, wenn der MR länger als eine Woche ohne Einwände offen ist merge ich ihn.
Wenn du einen erstellt hast dann mach bitte eine eMail auf der Mailingliste und stelle diesen zur Abstimmung/CodeReview oder ähnliches und geb Leuten Zeit darauf zu reagieren!
Da bin ich gegen. Du hast hast jederzeit die Möglichkeit den Code ein zu sehen und spätestens bei einem SignRequest Einwände zu äußern.
Eine Alternative die ich sehr bevorzugen würde, wäre ein Patchworker einzureichen. Dann arbeiten wir nur über Patches auf der ML da würde ich die MR Funktion allerdings im git deaktivieren.
Also du hast das Thema mal angesprochen Ja aber eine eindeutige Entscheidungsfindung gab es nie
Das Thema stand Lange auf der ML zur Diskussion wenn keine Einwände kommen ist davon auszugehen das alle damit einverstanden sind.
vg Tarek