Skip to content

Docs: Rename CONTEXT structure files to fix URL encoding routing issues - #2223

Merged
John Kennedy (GrantMeStrength) merged 1 commit into
MicrosoftDocs:docsfrom
natitati4:fix-context-links
Sep 14, 2026
Merged

John Kennedy (GrantMeStrength) merged 1 commit into
MicrosoftDocs:docsfrom
natitati4:fix-context-links

Conversation

@natitati4

@natitati4 Netanel Komm (natitati4) commented Jun 15, 2026 •

Copy link
Copy Markdown
Contributor

The architecture table used relative links with tildes (like ns-winnt-context~r2.md). While these resolve in GitHub, the live Microsoft Learn build system seems to convert tildes in file names to hyphens, but doesn't update the relative markdown links, resulting in links that lead to 404 pages on the live site.

Renamed the files to use hyphens (-r1, -r2) and updated the cross-references in the x86 and ARM context documentation.

@natitati4

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

@prmerger-automator

Copy link
Copy Markdown

Netanel Komm (@natitati4) : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

The architecture table used relative links with tildes (like ns-winnt-context~r2.md).
While these resolve in GitHub, the live Microsoft Learn build system seems to convert
tildes in file names to hyphens, but doesn't update the relative markdown links,
resulting in links that lead to 404 pages on the live site.

Renamed the files to use hyphens (-r1, -r2) and updated the cross-references
in the x86 and ARM context documentation.

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.

LGTM — accurate technical correction. Approved.

@GrantMeStrength

Copy link
Copy Markdown
Contributor

Copilot /review

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.

Reviewed ✅ — Documentation improvement looks good.

@natitati4

Copy link
Copy Markdown
Contributor Author

Thanks for the approval, is there anything else needed in order to merge?

@natitati4

Copy link
Copy Markdown
Contributor Author

@prmerger-automator

Copy link
Copy Markdown

Netanel Komm (@natitati4) : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

Copilot AI 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.

🟢 Approval recommended

The renamed files exist, all affected links are consistent, and no obsolete references remain.

Pull request overview

Renames architecture-specific CONTEXT documentation files to avoid Microsoft Learn URL routing failures and updates all cross-references.

Changes:

  • Renames Arm32 and x86 32-bit pages to use hyphenated filenames.
  • Updates architecture-table links to the new paths.
File summaries
File Description
ns-winnt-context.md Updates x86 and Arm32 links.
ns-winnt-context-r2.md Updates the Arm32 link.
ns-winnt-context-r1.md Updates the x86 32-bit link.
ns-winnt-arm64_nt_context.md Updates x86 and Arm32 links.
Review details
  • Files reviewed: 4/4 changed files
  • Comments generated: 0
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@GrantMeStrength
John Kennedy (GrantMeStrength) merged commit 186a3a2 into MicrosoftDocs:docs Sep 14, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants