-
Notifications
You must be signed in to change notification settings - Fork 25.1k
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
Remove superfluous HTML start element at end of body #77563
Remove superfluous HTML start element at end of body #77563
Conversation
There is no test for this change, other than compiling with JDK 17 - which completes successfully on "my machine" ;-). |
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.
LGTM
Pinging @elastic/es-core-infra (Team:Core/Infra) |
Pinging @elastic/es-docs (Team:Docs) |
@elasticmachine update branch |
💔 Backport failed
To backport manually run |
In JDK 17, doclint invoked by javac warns of constructs that lead to empty paragraphs in the generated documentation. We have a handful of these, redundant use of
<p>
at the end of a block of text, which trips up the build. This a preparatory change intended to help smoothen the eventual transition to 17.The JDK issue that introduced this stricter doclint check in 17 is https://bugs.openjdk.java.net/browse/JDK-8258957.