Skip to content

docs: use direct same-page xref for Ignored Files Editor (7.1)#741

Merged
phil-davis merged 1 commit into
7.1from
fix/727-ignored-files-xref-7.1
Jun 22, 2026
Merged

docs: use direct same-page xref for Ignored Files Editor (7.1)#741
phil-davis merged 1 commit into
7.1from
fix/727-ignored-files-xref-7.1

Conversation

@DeepDiver1975

Copy link
Copy Markdown
Member

What

Replaces the alias-routed xref at using.adoc:630 with a direct same-page xref: navigating.adoc#using-the-ignored-files-editorusing-the-ignored-files-editor.

Why

navigating.adoc is only a page-aliases entry of using.adoc itself, so this was a same-page link routed through the page's own alias — brittle and inconsistent. The target heading ==== Using the Ignored Files Editor (using.adoc:570) exists. This matches the existing correct same-page xref at using.adoc:538.

Refs #727 (master PR carries the closing keyword).

🤖 Generated with Claude Code

The xref routed through `navigating.adoc#using-the-ignored-files-editor`,
but `navigating.adoc` is only a page-aliases entry of using.adoc itself.
This is a brittle self-referential link through the page's own alias.
Replace it with a direct same-page xref, matching the existing correct
reference elsewhere on the page (using.adoc:538).

Closes #727

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>

@DeepDiver1975 DeepDiver1975 left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

🤖 Automated code review by Claude Code review agent

Approve. 7.1 port using a direct same-page xref for the Ignored Files Editor (xref:using-the-ignored-files-editor[...]).

  • Target heading ==== Using the Ignored Files Editor exists in using.adoc; the new xref resolves and matches the existing correct same-page xref in the file. navigating.adoc is a self-alias of using.adoc, confirming the old indirection was unnecessary.
  • AsciiDoc syntax correct.

Pair consistency: byte-identical change to master PR #742, correctly targets the 7.1 branch, uses Refs #727. LGTM.

@phil-davis phil-davis merged commit c85d7d5 into 7.1 Jun 22, 2026
1 check passed
@phil-davis phil-davis deleted the fix/727-ignored-files-xref-7.1 branch June 22, 2026 09:18
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.

2 participants