-
Notifications
You must be signed in to change notification settings - Fork 18
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
Jameica und Hibiscus Abhängigkeit im build definieren und bauen #677
Jameica und Hibiscus Abhängigkeit im build definieren und bauen #677
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Step "restore-cache" nach reusable-build.yml
auslagern.
ae7e329
to
3b54e0d
Compare
Es ist auf jeden Fall schon mal cool, dass die Cache Usage mit diesem PR mehr als halbiert wird (siehe https://github.com/openjverein/jverein/actions/caches). Dadurch laufen die Actions auch schneller. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sehr schön aufgeräumt sieht es nun aus. Vielen Dank und entschuldige, dass ich da bisl nervig war.
Ich kenne mich bisher mit den Actions nur wenig aus und überblicke daher nicht alle folgen. Wenn sich sonst niemand findet kann ich es schon aprooven. @dippeal ich glaube du kennst dich besser damit aus und ich fände gut wenn du es reviewen könntest. |
Die Version von Jameica war noch alt. Die habe ich jetzt auf die aktuellste angepasst. |
Kannst du mal bitte skizzieren oder erklären, wo der renovate ins Spiel kommt bei dem PR? |
Mit diesem pr existiert eine Versionsnummer, die von renovate verwendet werden kann. Bis jetzt nehmen wir einfach was gerade aktuell ist. Renovate muss noch kommen, siehe #699 . Der cache wird durch die cache action befüllt. Die schaut, ob sie einen miss hatte und aktualisiert im Nachgang den cache mit dem cache key. Der key generiert sich aus den build.properties und build.xml Dateien. Wenn diese sich geändert haben, ist nicht mehr sicher, dass wir die gleichen Abhängigkeiten haben. |
Da der Umbau auf Maven wohl doch ein zu großer erster Schritt ist, habe ich den ant build erweitert. Mit dem target
build-dependencies
werden jameica und hibiscus heruntergeladen und gebaut, so wie es der jverein build erwartet. Dies funktioniert lokal wie im Github Action Kontext identisch. Somit kann das Aufsetzen des lokalen builds vereinfacht werden und man baut gegen gleiche Versionen wie in Github. In einem zweiten Schritt kann renovate aktiviert werden, so dass neue Versionen erkannt und ein pr erstellt wird.Die workflows sind auch wesentlich vereinfacht, da der cache nun abhängig von build.properties und build.xml ist. Solange sich dort nichts verändert, haben sich auch keine externen Abhängigkeiten verändert. Es werden nur die Elemente gecached, die auch im ant build im classpath referenziert werden.