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

Asciidoctor: Implement beta and experimental #662

Merged
merged 4 commits into from
Mar 5, 2019
Merged

Conversation

nik9000
Copy link
Member

@nik9000 nik9000 commented Mar 5, 2019

Implements the beta[] and experimental[] macros for asciidoctor. It
adds integration tests that make sure that we copy the required image
even if that is the only admonition in the book. This is an especially
good idea here because the code that copies the images for the
admonitions is a little sneaky.

nik9000 added 2 commits March 4, 2019 18:26
Implements the `beta[]` and `experimental[]` macros for asciidoctor. It
adds integration tests that make sure that we copy the required image
even if that is the only admonition in the book. This is an especially
good idea here because the code that copies the images for the
admonitions is a little sneaky.
@lcawl
Copy link
Contributor

lcawl commented Mar 5, 2019

Thanks, this fixes beta[] for the Stack Overview (#649)

README.asciidoc Outdated
////
This file contains the sequence `_` to escape from around Elastic's
Asciidoctor plugin that provides Asciidoc compatibility. It automatically
translates things like `beta[]` into their Asciidoctor compatible for
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/for/form/

@nik9000 nik9000 requested a review from ninaspitfire March 5, 2019 14:00
@nik9000
Copy link
Member Author

nik9000 commented Mar 5, 2019

Thanks for reviewing @jarpy!

@nik9000 nik9000 merged commit 7d243dd into elastic:master Mar 5, 2019
@nik9000 nik9000 mentioned this pull request Mar 5, 2019
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.

3 participants