-
Notifications
You must be signed in to change notification settings - Fork 25k
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
Docs: Rework close ml section #41435
Conversation
Neat! I didn't see that failure locally. |
Reworks the close ml section to work properly in Asciidoctor. It renders a little funny in AsciiDoc but AsciiDoc is on its way out anyway....
I didn't commit one part of my original change..... |
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 and builds successfully.
As @nik9000 noted, the rendering in the Asciidoc version is a little wonky but doesn't break functionality.
OOh. That extra +.... |
Oh well. I can live with it for the next few days until we do the conversion. I'm hoping it'll be days. |
Reworks the close ml section to work properly in Asciidoctor. It renders a little funny in AsciiDoc but AsciiDoc is on its way out anyway....
Reworks the close ml section to work properly in Asciidoctor. It renders a little funny in AsciiDoc but AsciiDoc is on its way out anyway....
I've backported this. master and 7.0 can now be built with asciidoctor. 7.x is still busted. I'll see if I can fix it. |
Reworks the close ml section to work properly in Asciidoctor. It renders a little funny in AsciiDoc but AsciiDoc is on its way out anyway....
Reworks the close ml section to work properly in Asciidoctor. It
renders a little funny in AsciiDoc but AsciiDoc is on its way out
anyway....
Relates to #41128