Repository navigation
docs: fix all broken docs.nvidia.com links in root README - #2399
sven-rankov wants to merge 2 commits into
Conversation
The documentation site was reorganized and several root README links still pointed at the old URL scheme, returning 404. Update the six affected destinations to their current locations: - Colang 2.0 guide -> configure-guardrails/colang/colang-2 - Guardrails library (two links) -> configure-guardrails/guardrail-catalog - Using Docker -> more-deployment-options/using-docker - LangChain integration -> integration-with-third-party-libraries/langchain/langchain-integration - Security Guidelines -> resources/guidelines - Telemetry -> resources/telemetry Each replacement URL was verified to return HTTP 200. The "Getting Started Guide" link is tracked separately in NVIDIA-NeMo#2381 and is intentionally left unchanged here to keep the scopes from overlapping. Signed-off-by: Tsvetan Rankov <tsvetan.rankov@gmail.com>
8d12fe1 to
a18a9cb
Compare
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: NVIDIA-NeMo/Guardrails/.coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review. 📝 WalkthroughWalkthroughThe README links for Colang 2.0, the guardrails catalog, Docker, LangChain, security guidelines, and telemetry now point to updated documentation paths. The LangChain integration instructions remain unchanged. ChangesREADME documentation links
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Other · Severity of issue fixed: Low Merge Risk: ⚪ Minimal · up to The README link updates have no established issue in the supplied review context; the reported URL and README-check results are consistent with merging. 🚥 Pre-merge checks | ✅ 6✅ Passed checks (6 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
[Low impact] The PR appears safe to merge; no actionable issues were found. SummaryUpdates six documentation destinations across seven links in
Reviews (2) · Last reviewed commit: "ci: retrigger PR checks" · Reviewed by Greptile |
miyoungc
left a comment
There was a problem hiding this comment.
Reviewed with assistance from Codex. All six replacement documentation URLs resolve to the intended pages and match the documentation navigation. Changed-file pre-commit, including ty, and git diff --check passed at a18a9cb. No actionable findings.
Signed-off-by: Miyoung Choi <miyoungc@nvidia.com>
PR merge guidance@sven-rankov thanks for the PR. GitHub is currently blocking merge for one or more repository requirements:
Relevant guide: |
Description
The documentation site was reorganized and six link destinations in the root
README still point at the old URL scheme, returning 404. This updates them to
their current locations. Text and link labels are unchanged — only the URLs move.
Scope note: the "Getting Started Guide" link is also broken, but it is already
tracked in #2381, so it is deliberately left untouched here to avoid overlapping
with that issue.
Related Issue(s)
Fixes #2395
Issue assignee: (not yet assigned)
Verification
curl -sL -o /dev/null -w '%{http_code}').docs.nvidia.comlink in the README (16 unique): after this changethe only remaining 404 is the
getting-startedlink tracked in doc: Broken 'Getting Started Guide' link in root README #2381.pre-commit run --files README.mdpasses.AI Assistance
Checklist
Summary by CodeRabbit