Skip to content

Conversation

wangshangsam
Copy link
Contributor

@wangshangsam wangshangsam commented Sep 4, 2025

What does this PR do ?

This PR adds a converter (for the Sphinx documentation) that converts "the relative path to a file" in the Markdown to "the URL of the file on GitHub" being displayed on the documentation website.

This PR also fixes minor coding styles issues from a previous PR (#837).

Issues

Fixes #577

Usage

Screencast.from.2025-09-03.06.59.58.PM.webm

Before your PR is "Ready for review"

Pre checks:

  • Make sure you read and followed Contributor guidelines
  • Did you write any new necessary tests?
  • Did you run the unit tests and functional tests locally? Visit our Testing Guide for how to run tests
  • Did you add or update any necessary documentation? Visit our Document Development Guide for how to write, build and test the docs.

Additional Information

  • ...

@wangshangsam wangshangsam self-assigned this Sep 4, 2025
@wangshangsam wangshangsam added bug Something isn't working documentation Improvements or additions to documentation labels Sep 4, 2025
wangshangsam and others added 8 commits September 4, 2025 15:53
Signed-off-by: Shang Wang <samshang.wang@mail.utoronto.ca>
)

Signed-off-by: ashors1 <ashors@nvidia.com>
Signed-off-by: Shang Wang <samshang.wang@mail.utoronto.ca>
Signed-off-by: Charlie Truong <chtruong@nvidia.com>
Signed-off-by: Shang Wang <samshang.wang@mail.utoronto.ca>
Signed-off-by: Terry Kong <terryk@nvidia.com>
Signed-off-by: Shang Wang <samshang.wang@mail.utoronto.ca>
Signed-off-by: Zhiyu Li <zhiyul@nvidia.com>
Signed-off-by: Zhiyu Li <zhiyul@NVIDIA.com>
Co-authored-by: Yuki Huang <48991475+yuki-97@users.noreply.github.com>
Signed-off-by: Shang Wang <samshang.wang@mail.utoronto.ca>
Signed-off-by: Terry Kong <terryk@nvidia.com>
Signed-off-by: Shang Wang <samshang.wang@mail.utoronto.ca>
Signed-off-by: Shang Wang <samshang.wang@mail.utoronto.ca>
Signed-off-by: Shang Wang <samshang.wang@mail.utoronto.ca>
@wangshangsam wangshangsam force-pushed the wangshangsam/fix-rel-src-link branch from c08bbde to 6052183 Compare September 4, 2025 19:53
@github-actions github-actions bot added the CI Relating to CI label Sep 4, 2025
@github-actions github-actions bot removed the CI Relating to CI label Sep 4, 2025
Copy link
Contributor

@terrykong terrykong left a comment

Choose a reason for hiding this comment

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

thanks @wangshangsam . a much needed QoL improvement. I just had a small comment

Signed-off-by: Shang Wang <samshang.wang@mail.utoronto.ca>
Signed-off-by: Shang Wang <samshang.wang@mail.utoronto.ca>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace relative source links with github links
5 participants