Skip to content

Remove 72 redundant anchor tags#8209

Merged
dereuromark merged 1 commit into5.xfrom
fix-docs-anchors-and-example
Jan 30, 2026
Merged

Remove 72 redundant anchor tags#8209
dereuromark merged 1 commit into5.xfrom
fix-docs-anchors-and-example

Conversation

@dereuromark
Copy link
Copy Markdown
Member

Summary

  • Remove <a id="..."></a> tags where the anchor ID is identical to what the heading auto-generates
  • For example, <a id="saving-entities"></a> before ## Saving Entities is redundant since the heading produces the same #saving-entities anchor
  • 72 anchors removed across 33 files
  • 131 anchors with differing IDs are preserved for legacy link compatibility (e.g. <a id="table-find-list"></a> before ## Finding Key/Value Pairs)

Test plan

  • Verify documentation builds without errors
  • Spot-check that internal cross-references still resolve correctly

@dereuromark dereuromark requested a review from josbeir January 30, 2026 09:37
Remove <a id="..."></a> tags that duplicate the anchor ID already
auto-generated by the markdown processor from the heading text.
For example, <a id="saving-entities"></a> before ## Saving Entities
is redundant since the heading produces the same #saving-entities
anchor. Anchors with IDs that differ from the heading's auto-generated
ID are preserved as they provide legacy link compatibility.
@dereuromark dereuromark force-pushed the fix-docs-anchors-and-example branch from 616b675 to 12177fb Compare January 30, 2026 09:52
@dereuromark
Copy link
Copy Markdown
Member Author

@josbeir We might want to have some validation for this.

@dereuromark dereuromark merged commit 1a334c7 into 5.x Jan 30, 2026
5 checks passed
@dereuromark dereuromark deleted the fix-docs-anchors-and-example branch January 30, 2026 09:58
@josbeir
Copy link
Copy Markdown
Contributor

josbeir commented Jan 30, 2026

@josbeir We might want to have some validation for this.

Normally the link checker checks if the hashes exist but maybe that needs some optimization. The script is in the bin folder...

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