-
Notifications
You must be signed in to change notification settings - Fork 197
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] Clean up cross-repo links #6557
[docs] Clean up cross-repo links #6557
Conversation
A documentation preview will be available soon. Request a new doc build by commenting
If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here. |
This pull request does not have a backport label. Could you fix it @colleenmcginnis? 🙏
NOTE: |
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 for taking care of these links, @colleenmcginnis! I did some quick spot-checks and only noticed a few minor things. In addition to the comments that I left below, there is one other link that's not properly resolving in the preview:

Looks like a parentheses is missing from the end of the URL and we might need to move the text before the URL, e.g.:
[Date Math format](https://www.elastic.co/guide/en/elasticsearch/reference/current/common-options.html#date-math)
Feel free to push a commit here or open another PR to fix other syntax issues! |
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.
I can open a separate PR to fix the refs to the ES date time page. Thanks again for updating these links!
Follow up to #6554
When using cross-repo links, the path should be relative to the
docset.yml
not the full path within the repo (updated docs-builder docs).