[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: |
nastasha-solomon
left a comment
There was a problem hiding this comment.
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! |
nastasha-solomon
left a comment
There was a problem hiding this comment.
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.ymlnot the full path within the repo (updated docs-builder docs).