Backport #8424 #8494: Publish API docs to docs.nvidia.com - #8525
Conversation
Contributes to NVIDIA#8423 Replace the (currently unused) hard-coded `version-map` with one we can set via `vars`. This decouples updating the `versions.json` that powers the docs version switcher from code changes, making the docs release process simpler. N.B. I'll be updating the docs publishing shared workflow to use this version map, but it currently doesn't. Authors: - Joséphine Wolf Oberholtzer (https://github.com/josephine-wolf-oberholtzer) Approvers: - Bradley Dice (https://github.com/bdice) URL: NVIDIA#8494
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 12 included reviews per hour; 9 remain after this review. 📝 WalkthroughSummary by CodeRabbit
WalkthroughThe build and pull request workflows add API documentation publication after ChangesAPI documentation publishing
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This PR updates documentation publishing workflows for the 26.08 release branch. No actionable merge-blocking risk remains based on the supplied evidence. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.) ✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Comment |
02caedb
into
NVIDIA:release/26.08
Backports #8424 and #8494 to the release/26.08 branch so we can get 26.08 docs onto docs.nvidia.com.