Workflows um Format Stage erweitern #175
5 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: liquid-development/game-tracker#175
Reference in New Issue
Block a user
Delete Branch "setup/44-workflows-um-format-stage-erweitern"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Zugehörige Issue(s)
Closes #44
Closes #53
Beschreibung
Die Push-Workflow wurde um einen Formatierungsschritt und einen Schritt zur Versionserhöhung erweitert
Änderungen
Push-Pipeline
Die Push-Pipeline hat jetzt folgende Workflows:
developwird diepatch-Version erhöht (Bsp.:0.0.5->0.0.6)main&developwird der Code formattiert. Die Pipeline schlägt fehl, wenn das Formatieren nicht geklappt hat.main&developwird der Code getestet. Die Pipeline schlägt fehl, wenn mind. ein Test fehlschlägt.Pull-Request-Pipeline
Die Pull-Request-Pipeline hat weiterhin folgende Workflows:
Zusätzliche Anmerkungen
Genutze Action für Versionserhöhung: https://github.com/marketplace/actions/update-pubspec-version
WIP: Workflows um Format Stage erweiternto Workflows um Format Stage erweitern(quelle gemini also vielleicht teilweise bullshit, aber klingt plausibel)
@@ -49,2 +44,2 @@git commit -m "Actions: Auto-formatting [skip ci]"git pushapt-get updateapt-get install -y jqwofür brauchst du jq? gemini sagt ist unnötig
Das war schon vorher drin, und das hatte ohne nicht funktioniert. Ich kann gleich nochmal prüfen
@@ -51,0 +112,4 @@git config --global user.email "actions@yannick-weigert.de"git add pubspec.yamlgit commit -m "Updated version number [skip ci]"git push origin HEAD:${{ gitea.head_ref }}(Mein kommentar placement hat keine relevanz)
Was hältst du davon wenn man die Version erst erhöht, wenn tests & lint erfolgreich waren? also dass die needs test und format sind, wobei doch eigentlich tests und lint auch unnötig sind? ich mein wenn die fehlschlagen, kann doch nicht gemerget werden und push.yaml läuft doch erst nach merge?
und:
Gitea unterstützt caching, also könnte man die flutter dependencies cachen um zeit zu sparen, das gleiche bei pull request natürlich.
Könntest ne eigene action zur flutter installation benutzen und dann diesen cache nutzen https://github.com/subosito/flutter-action
Also die Flutter Action funktioniert auf Gitea nicht, das hab ich schon geprüft.
Dependencies cachen, ja okay, aber das dauert reel 5 Sekunden oder so. Was ich gerne Cachen würde wäre die Flutter installation, falls du da n besseren weg findest, das dauert nämlich am längsten
ja ich meine mit dependencies flutter auch
Ja aber wenn du sagst das geht nur mit der Flutter Action dann ist das raus, ich hab das versucht aber wenn ich die implementiere funktionioert der Flutter installationsprozess nicht
https://gitea.com/actions/cache haste dir das angeguckt?
und was ist mit dem ersten teil von meinem comment?
Ja also checke was du meinst, aber eig auch irrelevant. Weil Format kannst du ja nur auf einer Branch machen, wo du selber nicht aktiv arbeitest. Und man kann ja auch irgendwann drüber nachdenken, ob man die Checks in der PR optional macht und nicht zwingend brauch (z.B. format).
Versionserhöhung hab ich nur ans ende gemacht, damit es da keine konflikte gibt, wenn die
pubspec.yamlformatiert wird.@@ -13,4 +13,3 @@import 'package:game_tracker/presentation/widgets/tiles/group_tile.dart';import 'package:game_tracker/presentation/widgets/top_centered_message.dart';import 'package:provider/provider.dart';newline? hahahahahaah
Achso ja das war zum testen vom formattieren, fix ich
Workflows um Format Stage erweiternto WIP: Workflows um Format Stage erweiternWIP: Workflows um Format Stage erweiternto Workflows um Format Stage erweitern