Upload new builds as gh release #90
Conversation
|
oh krass, sehr cool. Wenn die Build-Prozesse weiterhin richtig laufen, können wir das gerne übernehmen. |
|
Die Releases sind unabhängig vom Branch, der aktuelle Trigger ist nur der Tag. Dann müsste ich nur schauen dass ich den PR auf den beta Branch umbiege bzw. einen neuen PR gegen den Branch machen, soll ich das tun? |
|
Ich glaube das wäre gut. Vielleicht reicht es ja den target-branch direkt hier im PR zu ändern. |
There was a problem hiding this comment.
Hab noch ein paar comments gemacht, wenn das noch geändert wird ists echt super :) 👍 Vielen Dank ! @lukasbach
| python -m pip install --upgrade pip | ||
| pip install -r requirements.txt | ||
| - name: Build Windows | ||
| run: pyinstaller --clean --noconfirm specs/windows-terminservice.spec --add-data "tools/cloudscraper;./cloudscraper/" |
There was a problem hiding this comment.
bitte den --add-data command in die .spec erstellung hinzufügen. (und in der Readme ändern). Damit kann dann auch der python code aus der .spec raus. Einfach nocheinmal die .spec mit dem --add-data command für tools/cloudscraper ausführen. (Linux und Win)
There was a problem hiding this comment.
Sorry, kannst du das nochmal ausführen, ich verstehe nicht ganz was du meinst. Pyinstaller wird hier ja schon mit tools/cloudscraper ausgeführt, was genau soll hier noch geändert werden?
.github/workflows/deploy.yaml
Outdated
| python -m pip install --upgrade pip | ||
| pip install -r requirements.txt | ||
| - name: Build Windows | ||
| run: pyinstaller --clean --noconfirm specs/windows-terminservice.spec --add-data "tools/cloudscraper;./cloudscraper/" |
There was a problem hiding this comment.
bitte den --add-data command in die .spec erstellung hinzufügen. (und in der Readme ändern). Damit kann dann auch der python code aus der .spec raus. Einfach nocheinmal die .spec mit dem --add-data command für tools/cloudscraper ausführen. (Linux und Win)
das gleiche natürlich hier :)
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
Ich würde sagen es wird Zeit, hier voranzukommen. Mein |
|
@JuliusJacobitz Vielen Dank für das Feedback. Bin aktuell mit einigem anderen beschäftigt und bin deswegen noch nicht dazu gekommen, bis spätestens morgen habe ich die Änderungen eingearbeitet. |
|
@lukasbach das passt :) reminder: es wird für Windows ein installer gebaut. Das kann dann als artifact verwendet werden. @haslersn ich hoffe deine Festplatte trägt das noch bis morgen :) |
@JuliusJacobitz Wo wird der Windows installer gebaut? Wenn ich pyinstaller mit der Windows spec ausführe, bekomme ich nur die entpackte app. Oder muss ich pyinstaller dafür anderst aufrufen? |
|
@JuliusJacobitz Habe die dokumentierenden Kommentare noch ergänzt, aber die Build logik noch nicht angepasst wegen der beiden Fragen die ich noch hatte. Kenne mich leider nicht so sehr mit dem Python Ökosystem und Pyinstaller aus 😅 |
Pyinstaller kann leider keine windows-installer bauen. Dafür wird im Moment "inno setup" verwendet. Der compiler führt das Script @lukasback kannst du das noch in die neuen workflows übernehmen und die conflicts lösen ? Gib gern bescheid wenn du noch Fragen hast :) Dann merge ich 👍 Edit: Oder soll ich das übernehmen ? :) |
|
Hab die Punkte noch ergänzt, müsste eigentlich passen jetzt, die pipeline scheint auch durchzulaufen. So sieht ein Release aus mit den Ergänzungen, da ist jetzt auch der Installer als Downloadable mitdrin: https://github.com/lukasbach/vaccipy/releases/tag/v1.0.1 @JuliusJacobitz gib Bescheid wenn noch was nicht stimmt. |
|
@lukasbach viele Dank für den Beitrag 👍 |
|
BIG UP! Danke an alle Beteiligten!! |
Ich schlage mal eine Änderung des Release-Prozesses vor, um die Zahl der Commits und die Größe des Repositories etwas überschaubarer zu halten :D
Gebt Bescheid wenn ich noch was ändern soll. Falls ihr den alten Prozess besser fandet, kann ich auch eine Ablehnung verstehen.
Meine Änderung entfernt den dist-Ordner aus dem repo, und ersetzt die alten Github action workflows mit zwei neuen Workflows:
Beispiel für ein Release: https://github.com/lukasbach/vaccipy/releases/tag/v1.0.0
Falls das eher gewünscht ist, kann ich die Pipeline auch anpassen, um bei jedem Commit auf master zu releasen, um den aktuellen Prozess eher abzubilden, ich persöhnlich finde es so aber besser da man so eher Kontroller über Releases hat.