Felix Kirchner
c10bc9dbdc
Some checks failed
Push Pipeline / test (push) Successful in 47s
Push Pipeline / update_version (push) Failing after 31s
Push Pipeline / generate_licenses (push) Failing after 36s
Push Pipeline / generate_localizations (push) Successful in 28s
Push Pipeline / sort_arb_files (push) Successful in 37s
Push Pipeline / format (push) Has been skipped
Push Pipeline / build (push) Successful in 5m32s
### Zugehörige Issue(s) Closes #222 ### Beschreibung Die Push & PR-Pipelines wurden um Überprüfungen und Formatierung für die Lokalisierung erweitert. Außerdem wurde eine PR-Checkliste zum PR-Template hinzugefügt. ### Änderungen - Template um `PR-Checklist` ergänzt (siehe Diff, gerne Ideen für mehr Punkte) - PR-Pipeline überprüft jetzt ob `untranslated_messages.json` leer (also `{}`) ist. Wenn nicht, schlägt sie fehl - In der Push-Pipeline werden jetzt die `.arb`-Files sortiert - In der Push-Pipeline werden jetzt die lokalisierungen generiert - Renovate Workflow hinzugefügt, welcher jede Woche die Dependencies überprüft (siehe [diese Test-PR](#244)) ### Zusätzliche Anmerkungen Ich wollt eigentlich auch einen Job zum generieren der DB-Files einbauen. Das hat jedoch nicht geklappt (siehe [hier](https://git.yannick-weigert.de/liquid-development/game-tracker/actions/runs/1013/jobs/3) oder [hier](https://git.yannick-weigert.de/liquid-development/game-tracker/actions/runs/1014/jobs/3)), weil der jedes mal mehr als 10 Minuten gedauert hat und dann getimeoutet ist  Co-authored-by: Gitea Actions [bot] <actions@yannick-weigert.de> Reviewed-on: #223 Co-authored-by: Felix Kirchner <felix.kirchner.fk@gmail.com> Co-committed-by: Felix Kirchner <felix.kirchner.fk@gmail.com>
65 lines
2.0 KiB
YAML
65 lines
2.0 KiB
YAML
name: Pull Request
|
|
about: Vorlage für Pull Requests
|
|
title: "WIP: [Name des Issues]"
|
|
|
|
body:
|
|
- type: input
|
|
id: related_issue
|
|
attributes:
|
|
label: Zugehörige Issue(s)
|
|
description: Issues welche mit diesem Pull Request geschlossen werden sollen
|
|
placeholder: "Closes #123"
|
|
validations:
|
|
required: true
|
|
|
|
- type: textarea
|
|
id: description
|
|
attributes:
|
|
label: Beschreibung
|
|
description: |
|
|
Eine klare und prägnante Zusammenfassung aller vorgenommenen Änderungen.
|
|
placeholder: |
|
|
Was wurde geändert?
|
|
validations:
|
|
required: true
|
|
|
|
- type: textarea
|
|
id: changes
|
|
attributes:
|
|
label: Änderungen
|
|
description: Liste alle Änderungen detailiert auf, die in diesem Pull Request vorgenommen wurden.
|
|
placeholder: |
|
|
- Neue Funktion X hinzugefügt
|
|
- Bug in Komponente X behoben
|
|
- Modul X für bessere Leistung refactored
|
|
- Dependencies aktualisiert
|
|
|
|
- type: checkboxes
|
|
id: checklist
|
|
attributes:
|
|
label: PR-Checkliste
|
|
description: Stelle sicher, dass alle Punkte erfüllt sind, bevor du den Pull Request zum Review freigibst.
|
|
options:
|
|
- label: Ich habe für Navigationen den AdaptiveNavigator` verwendet
|
|
required: false
|
|
- label: Ich habe alle Strings lokalisiert (de/en)
|
|
required: false
|
|
- label: Ich habe die Testdaten aktualisiert oder erweitert
|
|
required: false
|
|
- label: Ich habe das JSON-Schema angepasst
|
|
required: false
|
|
- label: Ich habe Tests für neue Datenbank/DAO-Funktionen hinzugefügt
|
|
required: false
|
|
|
|
- type: textarea
|
|
id: additional_notes
|
|
attributes:
|
|
label: Zusätzliche Anmerkungen
|
|
description: |
|
|
Gibt es zusätzlichen Kontext, Einschränkungen oder Informationen,
|
|
die Reviewer wissen sollten?
|
|
placeholder: |
|
|
- Bekannte Einschränkungen
|
|
- Offene TODOs
|
|
- Hinweise für Reviewer
|