Repository navigation
Conversation
added 13 commits
August 29, 2026 22:18
Contributor
|
Jeg syntes det er en god tanke, og veldig bra det blir godt dokumentert. |
Contributor
|
Kanskje litt på grensa å dytte inn mvn:set-version i set-enviroment action. set-version er eigentleg førebuing til å laga ein release... Det er ikkje nødvendig for å f.eks. køyre test på branch. |
Contributor
|
Se diskusjon her: https://digdir.slack.com/archives/C0B5QLATB47/p1788177777614949 |
oyri
reviewed
Oct 15, 2026
oyri
approved these changes
Nov 2, 2026
This reverts commit b3c9f91.
oyri
approved these changes
Nov 2, 2026
brasande
marked this pull request as ready for review
November 2, 2026 10:52
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR adds two new composite actions:
setup-maven-env– sets up JDK, Maven dependency caching and GitHub Packages credentialsmaven-run– runs Maven goals with support for mono-repos (application-path), multi-module repos (module-name), profiles, snapshot updates and extra argumentsThe golden path workflows now use these instead of duplicating the setup and Maven command logic.
Changes since the original PR from Jonas
main(incl. PF-2631 Støtte for unit-tests for Quarkus i pr-checks (ci-pr-checks-image) #507 Quarkus unit tests and PF-2590 Migrate to server-username-env-var/password in actions/setup-java #511setup-javamigration)setup-maven-envnow usessetup-javav6.0.0 withserver-username-env-var/server-password-env-var(aligned with PF-2590 Migrate to server-username-env-var/password in actions/setup-java #511)mvn versions:setmoved out ofsetup-maven-env– version setting stays as an explicit step in the publish workflows (Spring Boot image, Quarkus image, lib), since it is a release step and not environment setupapplication-pathinput fromsetup-maven-envBehaviour change
Spring Boot builds without
module-namenow runinstall spring-boot:build-image(previouslyspring-boot:build-image), so single-module and multi-module use the same command. No impact on tests or the resulting image.Type of Change
workflows to fail)
Checklist for Shared Workflows & Actions
(e.g., usage docs,
README.md, or inline workflow or action descriptions) toreflect these changes.
inputs, I have exposed them inall relevant proxy workflows or caller templates that rely on this workflow or
action.
applies to other shared workflows/actions in this repository and updated them
if necessary.
Testing Details
plattform-test-app (Spring Boot)
plattform-test-app-quarkus
eudiw-relying-party-register (Spring Boot mono-repo,
apps/eudiw-trustlist-service/) – tested in eudi-wallet-no/eudiw-relying-party-register#145Not tested
module-name) - skippedBefore merge
@main