-
Notifications
You must be signed in to change notification settings - Fork 25.6k
[DOC] Add X-Pack links to Elasticsearch Reference #25164
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
clintongormley
left a comment
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 (minor alignment issues)
docs/Versions.asciidoc
Outdated
| :plugins: https://www.elastic.co/guide/en/elasticsearch/plugins/{branch} | ||
| :javaclient: https://www.elastic.co/guide/en/elasticsearch/client/java-api/{branch} | ||
| :xpack: https://www.elastic.co/guide/en/x-pack/{branch} | ||
| :xpack-ref: https://www.elastic.co/guide/en/x-pack/{branch} |
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.
Please line up the URL with the ones before and after. You don't use tabs by any chance, do you? We have a build check that will cause tabs to fail the build.
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.
Thanks, I'll avoid tabs in the future!
docs/Versions.asciidoc
Outdated
| :es: Elasticsearch | ||
| :kib: Kibana | ||
|
|
||
| :xes-repo-dir: {docdir}/../../elasticsearch-extra/x-pack-elasticsearch/docs/en |
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.
Line the path up with the settings above
* [DOC] Add X-Pack links to Elasticsearch Reference * [DOCS] Address alignment of attributes in Versions.asciidoc
* [DOC] Add X-Pack links to Elasticsearch Reference * [DOCS] Address alignment of attributes in Versions.asciidoc
This pull request adds the infrastructure necessary to reference X-Pack content.
It uses a layout similar to what was worked out in the Kibana Reference for elastic/kibana#12062
It also adds some attribute definitions that are specific to X-Pack and changes the name of the attribute for the X-Pack Reference URL so that it does not cause confusion.