hpi-schul-cloud / nuxt-client

Schulcloud-Verbund-Software-Nuxt-Client
GNU Affero General Public License v3.0
32 stars 14 forks source link

Bulldozer (Bot) für automatisches Mergen, wenn möglich #39

Closed adrianjost closed 5 years ago

adrianjost commented 5 years ago

Ich habe diesen coolen Bot gefunden: https://github.com/palantir/bulldozer damit könnten wir automatisch mergen, sobald alles approved ist und die tests durchlaufen (wenn travis mal wieder länger braucht). Wenn es richtig konfiguriert ist könnte uns das eineiges an Arbeit abnehmen, da man nicht erst auf "update from master" klicken muss, 5min später wiederkommen muss um versuchen zu mergen und dann festzustellen, dass doch schon wieder was anderes im master gelandet ist.

adrianjost commented 5 years ago

Alternativen:

adrianjost commented 5 years ago

Ich habe etwas noch besseres gefunden: https://mergify.io/ Funktioniert direkt als Github app und muss nur konfiguriert werden. Ich evaluiere es gerade in einem privaten Projekt. Automatisches mergen, sobald alle Requirements erfüllt sind funktioniert schonmal, ich teste noch ob er auch automatisch "update from master" ausführen kann.

Config: https://github.com/adrianjost/Curriculum-Vitae/blob/master/.mergify.yml

adrianjost commented 5 years ago

@schul-cloud/coredevs Ich habe eine Anfrage gestellt, dass der Bot (https://github.com/apps/mergify) für dieses Repo aktiviert wird.

malcm commented 5 years ago

@schul-cloud/coredevs Wollt ihr das bei nuxt haben? ich denke für den produktiven Client und Server kann das unter Umständen zu Problemen kommen, wenn ein Feature drin ist mit dem keiner direkt gerechnet hat.

adrianjost commented 5 years ago

@malcm Ja, das ganze würde auch hinter einem label laufen, sodass der Bot erst anspringt wenn bspw. das Label "merge branch" gesetzt wurde. Das würde dann einiges an Zeit sparen, da man nicht regelmäßig gucken muss, ob endlich alle Tests durchgelaufen sind um zu mergen und ggf. erst nochmal vom master updaten muss und sich das Spiel wiederholt.

malcm commented 5 years ago

@adrianjost Klingt gut. Sollte dann aber noch in der readme beschrieben werden.

adrianjost commented 5 years ago

@malcm Definitiv! Kommt dann alles in unsere docs, pull templates usw.. Vielen Dank fürs aktivieren