Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

package.yml version anpassen #26

Open
staabm opened this issue Jun 19, 2022 · 4 comments
Open

package.yml version anpassen #26

staabm opened this issue Jun 19, 2022 · 4 comments

Comments

@staabm
Copy link
Member

staabm commented Jun 19, 2022

aktuell muss man immer manuell die versionnr in der package.yml erhöhen.
das kann man schnell vergessen und dann sind im installer mehrere downloads mit identischer versions nummern angabe

optimal fände ich, wenn die versionsnummer direkt aus dem github release gezogen werden würde und von der action in die package.yml geschrieben würde

@staabm
Copy link
Member Author

staabm commented Jun 20, 2022

Zufällig heute drüber gestolpert.. ggf. könnte man es so machen:

phpstan/phpstan-src@519d71f
(obacht, ggf. hat sich der workflow im nachhinein noch verändert)

@gharlan
Copy link
Member

gharlan commented Aug 23, 2022

das kann man schnell vergessen

So ist es heute auch bei https://github.com/FriendsOfREDAXO/yrewrite_domain_settings passiert.

Bei der automatischen Anpassung ist die Schwierigkeit, dass die package.yml ja eigentlich im Git-Tag bereits die neue Version haben sollte.
Bisher greift die Action aber ja erst nach finalem Release, also wenn Release und git-Tag schon vorhanden sind.

Vielleicht ist das im phpstan-Workflow gelöst, habe ich mir noch nicht genauer angeschaut. Allgemein müsste sich aber vermutlich das Vorgehen dann ändern, wenn wir eine automatische package.yml-Anpassung haben wollen.

Als ersten Schritt könnte man auch zumindest eine Validierung einbauen. Also dass die Action Releases ablehnt/bemängelt, wo der Tag nicht mit der Version in der package.yml übereinstimmt.

@staabm
Copy link
Member Author

staabm commented Aug 23, 2022

Als ersten Schritt könnte man auch zumindest eine Validierung einbauen. Also sodass die Action Releases ablehnt/bemängelt, wo der Tag nicht mit der Version in der package.yml übereinstimmt.

👍

@alxndr-w
Copy link
Member

Die Validierung ist bereits eingebaut und ich kann bestätigen, dass das einen recht erfolgreich vor der eigenen Vergesslichkeit schützt. Die automatische Anpassung der Version in der package.yml würde ich jedoch nicht bevorzugen. Ich empfinde es als sinnvoller, diese selbst festzulegen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants