-
Notifications
You must be signed in to change notification settings - Fork 342
Adds tech preview macro, updates experimental #2340
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
Conversation
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! Just a couple suggestions to keep things clear and tidy 🧹
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.
Left a minor comment about the docs. Otherwise, I agree with @gtback's comments.
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, aside from the issues already raised
Co-authored-by: Greg Back <[email protected]>
Co-authored-by: Greg Back <[email protected]>
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! Thanks @goodroot
@gtback Hi! The |
I'll 🧹 it up, @goodroot ! |
@gtback Any word? :) We wanna close this out soon. |
Sorry, I lost track of this one. I'll take a look now. |
@goodroot looks like there was some inconsistency between changes to the admonition text and the strings used in tests. I updated them and hopefully this version will pass. |
Finally! @goodroot feel free to merge when you're ready to. |
Thanks Greg :D I appreciate you. |
…tral-naming * upstream/master: (28 commits) Fixes .net book generation. (elastic#2367) Add main branch to elasticsearch-net repo to docs build. (elastic#2196) Update attributes.asciidoc (elastic#2365) Note that the full path is required for local doc build (elastic#2364) Bump current version to 7.17 (elastic#2333) moves old file back to bottom add new path for kibana links check apm: Neutral naming for apm-agent-java (elastic#2350) Change current to MS-68 (elastic#2355) Remove unnecessary branch mapping for package-spec (elastic#2359) Adds tech preview macro, updates experimental (elastic#2340) [DOCS] Add `plugins-7x` and `plugins-6x` attributes (elastic#2358) apm: Neutral naming for apm-agent-python (elastic#2348) apm: Neutral naming for apm-agent-rum-js (elastic#2354) apm: Neutral naming for apm-agent-dotnet (elastic#2352) apm: Neutral naming for apm-agent-nodejs (elastic#2353) Neutral naming for apm-agent-go (elastic#2351) apm: Neutral naming for apm-agent-php (elastic#2349) Add 7.last attribute for APM Guide (elastic#2356) [apm] main branch for apm-agent-ruby (elastic#2347) ...
Updates the old `experimental` admonition to use `preview` text when displayed inline. Relates to #2340
Adds a new macro for tech preview, updates old experimental macro to keep them both.
Closes #2305.