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
Modify link of install Elasticsearch #48943
Modify link of install Elasticsearch #48943
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.
Looking at the version of this page on master that is online, the link works for me and is pointing to https://www.elastic.co/guide/en/elasticsearch/reference/master/install-elasticsearch.html.
Would you mind sharing what you are seeing? I think the link is correct as is.
Pinging @elastic/es-docs (:Docs) |
Pinging @elastic/es-core-infra (:Core/Infra/Packaging) |
@cbuescher By the way, I have a question. Why do you use {es} attribution of asciidoc in reference/setup/install.asciidoc? If it is necessary to modify, I am going to submit a pull request. |
Thanks for asking, to be honest I don't know but @jrodewig might have an answer to that. I don't think it needs changing though. |
Hi @y-vectorfield! The We use attributes to stay DRY and consistent if we change a product name in the future. Updating the expanded term in a single place makes that much easier. It also reduces the likelihood of typos and misspellings. |
Hello, @jrodewig , |
Link of install.asciidoc is not correct. I modified this link.