-
Notifications
You must be signed in to change notification settings - Fork 41
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
Use GitHub setup-java action #187
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: sullis <[email protected]>
cc: @vietj |
PTAL @vietj |
ready for review |
what does it bring ? I would rather do that as part of the general maintenance of all of vertx github actions instead of just this one |
Details here:
|
open an issue here https://github.com/vert-x3/vertx-parent |
Issue is open |
Ready for review |
Motivation
joschi/setup-jdk
is obsolete. The action's repo was archived in December 2022The project README recommends using
actions/setup-java
Links
https://github.com/joschi/setup-jdk/blob/main/README.md
Modifications
replace
joschi/setup-jdk
with standard GitHub Actionsactions/setup-java
Conformance
Your commits should be signed and you should have signed the Eclipse Contributor Agreement as explained in https://github.com/eclipse/vert.x/blob/master/CONTRIBUTING.md
Please also make sure you adhere to the code style guidelines: https://github.com/vert-x3/wiki/wiki/Vert.x-code-style-guidelines