Skip to content

chore(beep boop 🤖): Bump uv.lock (main, mcore-main) (2026-03-26)#2996

Closed
svcnvidia-nemo-ci wants to merge 1 commit intomainfrom
bump-ci-container-2026-03-26-main-main
Closed

chore(beep boop 🤖): Bump uv.lock (main, mcore-main) (2026-03-26)#2996
svcnvidia-nemo-ci wants to merge 1 commit intomainfrom
bump-ci-container-2026-03-26-main-main

Conversation

@svcnvidia-nemo-ci
Copy link
Copy Markdown
Contributor

@svcnvidia-nemo-ci svcnvidia-nemo-ci commented Mar 26, 2026

🚀 PR to bump uv.lock in main.

🤖 This PR will be merged automatically once CI passes.

Summary by CodeRabbit

  • Chores
    • Updated submodule references to newer commits for improved project stability and access to latest upstream improvements.

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 7e29c20

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot bot commented Mar 26, 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 Mar 26, 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

Run ID: d30ba734-6903-439f-b408-8b6d0af6cd53

📥 Commits

Reviewing files that changed from the base of the PR and between 53d40d1 and 7e29c20.

⛔ 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 pull request updates Git submodule references for Megatron-LM. Both the .main.commit file and the 3rdparty/Megatron-LM submodule pointer are changed to reference commit 0528a40ddfe9e8cd71c8877f6f266fc01bdb576c, representing a version bump of the dependency.

Changes

Cohort / File(s) Summary
Submodule Version Bump
.main.commit, 3rdparty/Megatron-LM
Updated commit references from d45fa3e8e140fe5262179f6f94823869b89026a7 to 0528a40ddfe9e8cd71c8877f6f266fc01bdb576c for dependency version control.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

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

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title references bumping uv.lock but the actual changes involve updating .main.commit and a Git submodule reference for 3rdparty/Megatron-LM, not the uv.lock file itself. Update the title to accurately reflect the actual changes: updating commit references or submodule pointers, not the uv.lock file.
✅ 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 PR contains only minor dependency lock file and submodule reference updates generated by automated CI tooling, with no new features or breaking changes.

✏️ 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-03-26-main-main

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@yaoyu-33
Copy link
Copy Markdown
Contributor

Superseded by newer bot bump PR.

@yaoyu-33 yaoyu-33 closed this Mar 30, 2026
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.

3 participants