Skip to content

docs: fix broken links and small inconsistencies#2329

Merged
johntmyers merged 2 commits into
NVIDIA:mainfrom
mesutoezdil:docs/fix-links-and-consistency
Jul 20, 2026
Merged

docs: fix broken links and small inconsistencies#2329
johntmyers merged 2 commits into
NVIDIA:mainfrom
mesutoezdil:docs/fix-links-and-consistency

Conversation

@mesutoezdil

@mesutoezdil mesutoezdil commented Jul 16, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • README's GitHub Sandbox Tutorial link was missing the get-started segment, every other reference to that page uses the correct path
  • README's community sandboxes link pointed at a docs page that does not exist, replaced with the actual OpenShell-Community repo
  • README listed macOS/Windows/Linux as equally supported, but support-matrix.mdx marks Windows as experimental and macOS as Apple Silicon only
  • architecture/README.md's doc index was missing google-vertex-ai-provider.md
  • SECURITY.md had a mis-indented list item under the vulnerability report checklist
  • standardized on NVIDIA/OpenShell-Community casing for repo links, two docs used a lowercase variant

@mesutoezdil
docs: fix broken links and small inconsistencies
b1c3401 
- README: fix github-sandbox tutorial link missing get-started segment
- README: replace dead community-sandboxes doc link with the actual repo
- README: match supported host list to support-matrix.mdx
- architecture/README: list the missing google-vertex-ai-provider doc
- SECURITY.md: fix a mis-indented list item
- standardize on NVIDIA/OpenShell-Community casing for repo links
@copy-pr-bot

copy-pr-bot Bot commented Jul 16, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@pimlock

pimlock commented Jul 17, 2026

Copy link
Copy Markdown
Collaborator

/ok to test b32e982

@johntmyers
johntmyers added this pull request to the merge queue Jul 20, 2026
Merged via the queue into NVIDIA:main with commit 80987e9 Jul 20, 2026
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@krishicks krishicks krishicks approved these changes
@mrunalp mrunalp Awaiting requested review from mrunalp mrunalp is a code owner
@maxamillion maxamillion Awaiting requested review from maxamillion maxamillion is a code owner
@derekwaynecarr derekwaynecarr Awaiting requested review from derekwaynecarr derekwaynecarr is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@mesutoezdil @pimlock @krishicks @johntmyers