Skip to content

docs: fix broken observability and tokenizer links - #8413

Open
Xuanwo wants to merge 1 commit into
mainfrom
xuanwo/fix-docs-broken-links-8364-2058
Open

docs: fix broken observability and tokenizer links#8413
Xuanwo wants to merge 1 commit into
mainfrom
xuanwo/fix-docs-broken-links-8364-2058

Conversation

@Xuanwo

@Xuanwo Xuanwo commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

Fixes the two documentation 404s reported in #8364.

  • docs/src/guide/tokenizer.md: jieba-rs moved its dictionary into the jieba/ crate path (jieba/src/data/dict.txt).
  • docs/src/guide/observability.md: lance::metrics is feature-gated and docs.rs currently builds with no features, so docs.rs/.../lance/metrics/ 404s. Point the guide at the in-repo metrics.md source that the page already includes, and enable the metrics feature in [package.metadata.docs.rs] so the module is published after the next crate release.

Test plan

  • Confirmed both replacement URLs return HTTP 200
  • Scheduled docs link checker should clear the Errors entries for these two files once this lands on main

Fixes #8364

Open in Web Open in Cursor 

Point the Jieba dictionary example at the relocated path in jieba-rs,
link the metrics catalogue to the in-repo source that matches the
included snippet, and enable the metrics feature for docs.rs builds so
the module is published after the next crate release.

Fixes #8364
@cursor

cursor Bot commented Aug 7, 2026

Copy link
Copy Markdown

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@github-actions github-actions Bot added A-docs Documentation documentation Improvements or additions to documentation labels Aug 7, 2026
@Xuanwo
Xuanwo marked this pull request as ready for review August 7, 2026 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-docs Documentation documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docs link checker report

1 participant