Skip to content

Fix links in ADVANCED.md to use relative paths - #524

Merged
Jan-Kazlouski-elastic merged 1 commit into
elastic:mainfrom
insukcho:patch-1
Oct 22, 2026
Merged

Jan-Kazlouski-elastic merged 1 commit into
elastic:mainfrom
insukcho:patch-1

Conversation

@insukcho

@insukcho insukcho commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Fixing 4 broken links.

Checklists

Pre-Review Checklist

  • This PR does NOT contain credentials of any kind, such as API keys or username/passwords (double check crawler.yml.example and elasticsearch.yml.example)
  • This PR has a meaningful title
  • This PR links to all relevant GitHub issues that it fixes or partially addresses
    • If there is no GitHub issue, please create it. Each PR should have a link to an issue
  • this PR has a thorough description
  • Covered the changes with automated tests
  • Tested the changes locally
  • Added a label for each target release version (example: v0.1.0)

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

All four corrected links point to existing documentation files, with no unresolved issues.

No open issues 🎉

What changed in this PR

Fixes four broken links in the advanced crawler documentation by making their paths relative to docs/ADVANCED.md.

Changes:

  • Corrects crawl rules and crawler directives links in both the primary and purge crawl sections.
  • Changes documentation links only; crawler behavior is unchanged.
File Description
docs/​ADVANCED.md Removes the redundant docs/ prefix from four relative links.

Review effort: Balanced

@Jan-Kazlouski-elastic
Jan-Kazlouski-elastic enabled auto-merge (squash) October 22, 2026 07:37

@Jan-Kazlouski-elastic Jan-Kazlouski-elastic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your contribution @insukcho
Approved.

@Jan-Kazlouski-elastic

Copy link
Copy Markdown
Contributor

buildkite test this

@Jan-Kazlouski-elastic
Jan-Kazlouski-elastic merged commit aff4b1d into elastic:main Oct 22, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants