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

Fix quarkus gha #9

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Fix quarkus gha #9

wants to merge 7 commits into from

Conversation

jmtd
Copy link
Contributor

@jmtd jmtd commented Aug 4, 2023

Various small fix-ups to get the Quarkus GItHub action to function.

jmtd added 7 commits August 4, 2023 09:15
There are no registered git submodules for this repository (yet)

Signed-off-by: Jonathan Dowland <[email protected]>
We don't need it in the runner, if we're using S2I

Signed-off-by: Jonathan Dowland <[email protected]>
Separate out the build and push jobs so that we can trigger the build
stage in Pull Requests but not the Push.

Signed-off-by: Jonathan Dowland <[email protected]>
 * Don't define JAVA_HOME, inherit the value from the host.
 * Hard-code 17 instead of 11
 * Reference JAVA_HOME in INSTALL_BASE

Signed-off-by: Jonathan Dowland <[email protected]>
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

Successfully merging this pull request may close these issues.

1 participant