Skip to content

Fix broken source links in documentation#1934

Merged
ArthurZucker merged 1 commit intohuggingface:mainfrom
Shivam-Bhardwaj:fix-visualizer-docs-links
Mar 25, 2026
Merged

Fix broken source links in documentation#1934
ArthurZucker merged 1 commit intohuggingface:mainfrom
Shivam-Bhardwaj:fix-visualizer-docs-links

Conversation

@Shivam-Bhardwaj
Copy link
Copy Markdown
Contributor

What does this PR do?

Fixes broken GitHub source links in the API documentation (e.g., Visualizer page).

The doc-builder was generating links to src/tokenizers/... but the Python source files are located at bindings/python/py_src/tokenizers/....

Example broken link:
https://github.com/huggingface/tokenizers/blob/v0.20.3/src/tokenizers/tools/visualizer.py → 404

Correct path:
https://github.com/huggingface/tokenizers/blob/main/bindings/python/py_src/tokenizers/tools/visualizer.py

Fix

Added version_tag_suffix: bindings/python/py_src/ parameter to documentation build workflows to generate correct source links.

Fixes #1910

The documentation source links were pointing to `src/tokenizers/...` which
doesn't exist. The Python source files are located at
`bindings/python/py_src/tokenizers/...`.

Add `version_tag_suffix` parameter to documentation build workflows to
generate correct GitHub source links.

Fixes huggingface#1910
@ArthurZucker ArthurZucker merged commit e050211 into huggingface:main Mar 25, 2026
@ArthurZucker
Copy link
Copy Markdown
Collaborator

Ty

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.

[Docs] Visualizer dead links

2 participants