search

BreakOutEvent / breakout-frontend

Web Frontend
GNU Affero General Public License v3.0
6 stars 2 forks source link

Travis CI durch GitHub Actions ersetzen #430

Open sibbl opened 2 years ago

sibbl commented 2 years ago

Bei Travis haben wir "nur" 10.000 Build-Minuten. Diese sind bis 19.6. aufgebraucht.

D.h. bis dahin hätten wir erstmal nix deployen können da die Docker-Images über Travis gebaut wurden.

Als Ersatz nehmen wir GitHub Actions, die es mittlerweile gibt.

Stand:

sibbl commented 2 years ago

breakout-frontend ist fertig migriert.

Im Gegensatz zu vorher werden nur noch bei develop+master auch wirklich Docker-Container gebaut und gepushed. Das dürfte auch die Limits in unserer Azure Container Registry entlasten.

Die neue Methode kommt auch ohne verschlüsselte secrets.zip im Repository aus, die in dem Zuge entfernt wurde.

Was cool wäre: wenn wir irgendwann eslint updaten, könnte man die Fehler des Linter-Schrittes direkt mit in GitHub integrieren. War anfangs im Code mit drin, aber aufgrund unserer sehr veralteten eslint-Version nicht so einfach kompatibel.

sibbl commented 2 years ago

breakout-backend ist ebenso fertig.

Im Gegensatz zu vorher werden keine application*.properties mehr ins Docker-Image gebaut, die nicht auch im Git liegen. Diese sollten via Docker gemounted werden, sodass wir keine sensitiven Daten im Image liegen haben.

Was cool wäre: das Gradle-Test-Resultat noch mit in GitHub integrieren. Bin froh dass die Gradle Tests aber nun wenigstens überhaupt mal laufen. PS: die Datei anfangs application-test.properties zu nennen war eine schlechte Idee, weil dadurch gar kein Test mehr loslief.