chore(beep boop 🤖): Bump uv.lock (r0.4.0, mcore-core_r0.17.0) (2026-04-17)#3371
chore(beep boop 🤖): Bump uv.lock (r0.4.0, mcore-core_r0.17.0) (2026-04-17)#3371svcnvidia-nemo-ci wants to merge 1 commit intor0.4.0from
uv.lock (r0.4.0, mcore-core_r0.17.0) (2026-04-17)#3371Conversation
…-04-17) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
|
/ok to test 6de03f1 |
📝 WalkthroughWalkthroughThe pull request updates the Megatron-LM submodule reference by changing the tracked commit hash in both Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 3 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.main.commit:
- Line 1: The .main.commit file references a non-existent upstream commit hash
(d6b1813f2bfb867661b154574d76b3c0d395f426); replace it with a valid commit from
the upstream Megatron-LM repo or remove/refresh the file. To fix, fetch and
verify upstream commits (git fetch upstream; git rev-parse/upstream/main or git
ls-remote upstream) to obtain a reachable commit SHA, then update the value in
.main.commit to that verified SHA (or delete the file if it should not pin a
commit) and re-run your verification to ensure git no longer reports "not our
ref".
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: e3c68249-414a-44e5-8e58-75cb0e4f8553
⛔ Files ignored due to path filters (1)
uv.lockis excluded by!**/*.lock
📒 Files selected for processing (2)
.main.commit3rdparty/Megatron-LM
| @@ -1 +1 @@ | |||
| d7288711ba278d160d2a5a22c099915c9fe1395c | |||
| d6b1813f2bfb867661b154574d76b3c0d395f426 | |||
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Description: Verify the new commit hash exists in the Megatron-LM submodule
# Check if the commit exists in the submodule
cd 3rdparty/Megatron-LM
git cat-file -t d6b1813f2bfb867661b154574d76b3c0d395f426 2>/dev/null
if [ $? -eq 0 ]; then
echo "✓ Commit d6b1813f2bfb867661b154574d76b3c0d395f426 exists"
git log -1 --oneline d6b1813f2bfb867661b154574d76b3c0d395f426
else
echo "✗ Commit d6b1813f2bfb867661b154574d76b3c0d395f426 not found in submodule"
echo "Fetching latest changes..."
git fetch origin
git cat-file -t d6b1813f2bfb867661b154574d76b3c0d395f426
fiRepository: NVIDIA-NeMo/Megatron-Bridge
Length of output: 361
Do not merge: commit hash does not exist in upstream Megatron-LM repository.
The commit d6b1813f2bfb867661b154574d76b3c0d395f426 referenced in .main.commit cannot be resolved in the upstream Megatron-LM repository. Git returned the error: not our ref. This indicates the commit is not reachable and likely does not exist. Verify the commit hash is correct or obtain a valid commit from the upstream repository before proceeding.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In @.main.commit at line 1, The .main.commit file references a non-existent
upstream commit hash (d6b1813f2bfb867661b154574d76b3c0d395f426); replace it with
a valid commit from the upstream Megatron-LM repo or remove/refresh the file. To
fix, fetch and verify upstream commits (git fetch upstream; git
rev-parse/upstream/main or git ls-remote upstream) to obtain a reachable commit
SHA, then update the value in .main.commit to that verified SHA (or delete the
file if it should not pin a commit) and re-run your verification to ensure git
no longer reports "not our ref".
🚀 PR to bump
uv.lockinr0.4.0.🤖 This PR will be merged automatically once CI passes.
Summary by CodeRabbit