Skip to content

cp: fix(docs): remove absolute internal URL from 0.5.0 release notes (2397) into r0.5.0 - #2398

Merged
kajalj22 merged 1 commit into
r0.5.0from
cherry-pick-2397-r0.5.0
Aug 7, 2026
Merged

cp: fix(docs): remove absolute internal URL from 0.5.0 release notes (2397) into r0.5.0#2398
kajalj22 merged 1 commit into
r0.5.0from
cherry-pick-2397-r0.5.0

Conversation

@svcnvidia-nemo-ci

Copy link
Copy Markdown

beep boop [🤖]: Hi @kajalj22 👋,

we've cherry picked #2397 into  for you! 🚀

Please review and approve this cherry pick by your convenience!

## Summary

Fixes the `test_internal_pages_are_linked_by_path_not_by_absolute_url`
test failure introduced by #2389.

The v0.5.0 release notes sandboxing section contained a
`docs.nvidia.com/nemo/gym/...` absolute link. The test requires all
internal docs links in `latest/` to use relative paths (to keep readers
within the version they're reading). Replaced with a relative path
`/nemo-gym/nemo_gym/sandbox/providers` in both `latest/` and the
`v0.5.0/` snapshot.

## Test plan

- [ ] `test_internal_pages_are_linked_by_path_not_by_absolute_url`
passes
- [ ] Cherry-pick to `r0.5.0`

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Signed-off-by: Kajal Jain <kajalj@nvidia.com>
Co-authored-by: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: NeMo Bot <nemo-bot@nvidia.com>
@svcnvidia-nemo-ci

Copy link
Copy Markdown
Author

/ok to test 7c34638

@copy-pr-bot

copy-pr-bot Bot commented Aug 7, 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.

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

@kajalj22
kajalj22 merged commit 6e50df7 into r0.5.0 Aug 7, 2026
20 of 23 checks passed
@kajalj22
kajalj22 deleted the cherry-pick-2397-r0.5.0 branch August 7, 2026 00:11
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.

2 participants