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
ci: update workflows to Java 21 #3191
ci: update workflows to Java 21 #3191
Conversation
Co-authored-by: martinfrancois <f.martin@fastmail.com> Co-authored-by: georgebax <george.baxopoulos@gmail.com>
fd12898
to
53d3026
Compare
@grigala thanks for raising the PR! I've added more references to #3187 about the Javadoc stylesheet issues we encountered with the past upgrades. Do you plan to take care of it as well? It's basically about generating the Javadoc locally and checking if anything breaks in the resulting pages. If that's the case, we would need to figure out how to apply the same customizations to the Java 21 base stylesheet (like what I've done in #2621). |
Sure we are currently on Hackergarten event we will try to do it. Can you add me, @martinfrancois @georgebax as assignees? P.S. Greets from Hackergarten Basel team :) |
Co-authored-by: grigala <grigalaa@gmail.com>
Unfortunately, you were right, the CSS was broken again in Java 21... But we finally managed to fix it, see the last commit I pushed. |
It seems the search results lack highlighting. With the changes from this PR: As a minor point, we might want to remove the bold style from the SEARCH label/hyperlink. |
We would like to generate the Javadoc with Java 21 already in 3.25.0 so I'll have a look at the style for the search results. |
# Conflicts: # .github/workflows/cross-version.yml
This also makes the Javadoc stylesheet compatible with Java 21 and updates highlight.js from 11.5.1 to 11.9.0. Co-authored-by: martinfrancois <f.martin@fastmail.com> Co-authored-by: georgebax <georgebax@users.noreply.github.com> Co-authored-by: grigala <grigalaa@gmail.com> Co-authored-by: Stefano Cordio <stefano.cordio@gmail.com>
Thanks for your contribution, @grigala, @martinfrancois, and @georgebax! |
Check List:
Following the contributing guidelines will make it easier for us to review and accept your PR.