Skip to content

fix(docs): replace relative pathname:// links with full GitHub URLs - #3181

Open
Akhil Gupta (akhil9tiet) wants to merge 20 commits into
microsoft:mainfrom
akhil9tiet:fix/3180-pathname-docs-links
Open

Akhil Gupta (akhil9tiet) wants to merge 20 commits into
microsoft:mainfrom
akhil9tiet:fix/3180-pathname-docs-links

Conversation

@akhil9tiet

@akhil9tiet Akhil Gupta (akhil9tiet) commented Oct 10, 2026 •

Copy link
Copy Markdown

Pull Request

Description

Several docs pages linked to repository files outside the Docusaurus docs/ root with relative pathname:// links. Docusaurus emits these as plain relative hrefs, so on the published site they resolve under https://microsoft.github.io/hve-core/... where .github/ and evals/ content is not published — every one 404s. On GitHub.com the Markdown renderer drops pathname: links entirely.

Replaced each relative pathname:// link with a full https://github.com/microsoft/hve-core/blob/main/... URL (or tree/main/... for the directory link), matching the style most docs pages already use. 10 links fixed across 4 files. Also reformatted the Skills table in docs/agents/project-planning/README.md (row alignment broke with the longer URLs) and updated ms.date to 2026-10-10 on all 4 touched files per review.

Related Issue(s)

Fixes #3180

Type of Change

  • Documentation update

@akhil9tiet
Akhil Gupta (akhil9tiet) requested a review from a team as a code owner October 10, 2026 12:10
@codecov-commenter

Codecov Comments Bot (codecov-commenter) commented Oct 10, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.42%. Comparing base (49dfb73) to head (91c0743).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3181      +/-   ##
==========================================
+ Coverage   86.32%   86.42%   +0.10%     
==========================================
  Files          88      104      +16     
  Lines       11699    12007     +308     
  Branches        0       63      +63     
==========================================
+ Hits        10099    10377     +278     
- Misses       1600     1619      +19     
- Partials        0       11      +11     
Flag Coverage Δ
docusaurus 90.58% <ø> (?)
pester 86.32% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 17 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jkim323 Jamie Kim (jkim323) left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for picking this up and welcome! The link fixes look right: all 10 links from #3180 are covered, every target exists on main, and using tree/main for the directory link is correct. I have left a few comments for you to review below. Also, please update the PR description to follow our PR template and ensure the local validations are completed. Happy to review back once those are addressed!

Comment thread docs/agents/project-planning/README.md Outdated
Comment thread docs/agents/project-planning/README.md Outdated
Comment thread docs/agents/project-planning/arch-diagram-builder.md Outdated
Comment thread docs/customization/skills.md Outdated
Comment thread docs/planning/adrs/0011-define-vally-baseline-equivalence-evaluation-policy.md Outdated
@akhil9tiet

Copy link
Copy Markdown
Author

Jamie Kim (@jkim323) All 5 threads addressed: Skills table reformatted so rows align again, and ms.date updated to 2026-10-10 on all 4 files. PR description rewritten to follow the repo template. Ready for re-review.

@akhil9tiet

Copy link
Copy Markdown
Author

Jamie Kim (@jkim323) who else can review and approve this Pr, it says it requires 2 approvals.

This branch has not been deployed

No deployments
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.

fix(docs): relative pathname:// links to repository files 404 on the docs site

3 participants