Skip to content

chore(beep boop 🤖): Bump uv.lock (r0.4.0, mcore-core_r0.17.0) (2026-04-18)#3402

Open
svcnvidia-nemo-ci wants to merge 1 commit intor0.4.0from
bump-ci-container-2026-04-18-r0.4.0-core_r0.17.0
Open

chore(beep boop 🤖): Bump uv.lock (r0.4.0, mcore-core_r0.17.0) (2026-04-18)#3402
svcnvidia-nemo-ci wants to merge 1 commit intor0.4.0from
bump-ci-container-2026-04-18-r0.4.0-core_r0.17.0

Conversation

@svcnvidia-nemo-ci
Copy link
Copy Markdown
Contributor

@svcnvidia-nemo-ci svcnvidia-nemo-ci commented Apr 18, 2026

🚀 PR to bump uv.lock in r0.4.0.

🤖 This PR will be merged automatically once CI passes.

Summary by CodeRabbit

  • Chores
    • Updated Megatron-LM dependency to the latest version for improved performance and stability.

…-04-18)

Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@svcnvidia-nemo-ci
Copy link
Copy Markdown
Contributor Author

/ok to test ac3c817

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot bot commented Apr 18, 2026

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.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 18, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 5c8b4604-1e52-4325-9c44-6129a16a8d6f

📥 Commits

Reviewing files that changed from the base of the PR and between 9c2b3e4 and ac3c817.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (2)
  • .main.commit
  • 3rdparty/Megatron-LM

📝 Walkthrough

Walkthrough

This PR updates the Megatron-LM submodule pointer and corresponding commit reference from d7288711ba278d160d2a5a22c099915c9fe1395c to 779c4989c62af93371fb3d6d221faacfaaa91603. The changes are purely reference updates with no structural or functional modifications.

Changes

Cohort / File(s) Summary
Megatron-LM Submodule Update
.main.commit, 3rdparty/Megatron-LM
Updated submodule pointer and commit reference to newer Megatron-LM version.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • Megatron-Bridge#3352 — Updates the same Megatron-LM submodule pointer and .main.commit file.
  • Megatron-Bridge#2968 — Modifies identical files (.main.commit and 3rdparty/Megatron-LM submodule) to update Megatron-LM commit references.
  • Megatron-Bridge#3335 — Changes the 3rdparty/Megatron-LM git submodule pointer to a different commit hash.

Suggested reviewers

  • yaoyu-33
  • ko3n1g
🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title mentions 'Bump uv.lock' but the actual changes are to .main.commit and 3rdparty/Megatron-LM submodule pointer, not uv.lock files. Update the title to accurately reflect the changes: updating Megatron-LM submodule commit references, or verify if uv.lock changes are missing from the diff.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Test Results For Major Changes ✅ Passed Minor routine maintenance change updating submodule pointer with existing test infrastructure and CI validation.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bump-ci-container-2026-04-18-r0.4.0-core_r0.17.0

Comment @coderabbitai help to get the list of available commands and usage tips.

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