-
Notifications
You must be signed in to change notification settings - Fork 438
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update docs to use relative code links. #1447
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. LGTM with a minor fix.
Codecov Report
@@ Coverage Diff @@
## main #1447 +/- ##
=======================================
Coverage 85.31% 85.31%
=======================================
Files 154 154
Lines 4377 4377
=======================================
Hits 3734 3734
Misses 643 643 |
1186a8d
to
00a4ef9
Compare
docfx hates this apparently
|
Yes, I think we can fix |
Could we please try remove the prefix |
Use relative links to locations within the same repository. This means that in the Github UI the reader will follow the link to the location in the same branch rather than to a fixed, predetermined branch. Signed-off-by: James Peach <[email protected]>
507be83
to
28f5032
Compare
Updated to use root-relative links, and docfx seems happy. |
Changes
Use relative links to locations within the same repository. This means
that in the Github UI the reader will follow the link to the location
in the same branch rather than to a fixed, predetermined branch.