Skip to content

Ignore contributor DCO failures in Claude fix#5625

Merged
Phlip79 merged 2 commits into
NVIDIA:mainfrom
Phlip79:philip/ignore-claude-fix-dco
Jul 5, 2026
Merged

Ignore contributor DCO failures in Claude fix#5625
Phlip79 merged 2 commits into
NVIDIA:mainfrom
Phlip79:philip/ignore-claude-fix-dco

Conversation

@Phlip79

@Phlip79 Phlip79 commented Jul 2, 2026

Copy link
Copy Markdown
Member
  • I, the PR author, have personally reviewed every line of this PR.

What does this PR do?

/claude fix fails if previous commits are not passing DCO checks. In such cases, Claude should be ok with DCO failing. The author is the only person who can fix such a failure. The author will be mentioned in the comment.

Signed-off-by: Philip Petrakian <ppetrakian@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Jul 2, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

Signed-off-by: Philip Petrakian <ppetrakian@nvidia.com>
@Phlip79
Phlip79 marked this pull request as ready for review July 2, 2026 22:56
@Phlip79
Phlip79 requested a review from a team as a code owner July 2, 2026 22:56
@Phlip79

Phlip79 commented Jul 2, 2026

Copy link
Copy Markdown
Member Author

/ok to test 3d10138

@svcnvidia-nemo-ci svcnvidia-nemo-ci added the Approved All necessary approvals have been made label Jul 2, 2026
@Phlip79
Phlip79 enabled auto-merge July 2, 2026 23:20
@Phlip79
Phlip79 added this pull request to the merge queue Jul 3, 2026
@svcnvidia-nemo-ci

Copy link
Copy Markdown
Contributor

🔄 Merge queue validation started!

You can track the progress here: https://github.com/NVIDIA/Megatron-LM/actions/runs/28629882361

@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 3, 2026
@Phlip79
Phlip79 added this pull request to the merge queue Jul 5, 2026
@svcnvidia-nemo-ci

Copy link
Copy Markdown
Contributor

🔄 Merge queue validation started!

You can track the progress here: https://github.com/NVIDIA/Megatron-LM/actions/runs/28741695840

Merged via the queue into NVIDIA:main with commit 0823c73 Jul 5, 2026
93 of 95 checks passed
@Phlip79
Phlip79 deleted the philip/ignore-claude-fix-dco branch July 5, 2026 15:45
sidsingh-nvidia pushed a commit to sidsingh-nvidia/Megatron-LM that referenced this pull request Jul 9, 2026
Signed-off-by: Philip Petrakian <ppetrakian@nvidia.com>
Signed-off-by: Siddharth Singh <sidsingh@nvidia.com>
jon-barker pushed a commit to jon-barker/Megatron-LM that referenced this pull request Jul 10, 2026
Signed-off-by: Philip Petrakian <ppetrakian@nvidia.com>
Signed-off-by: Jon Barker <jbarker@aws-cmh-slurm-1-vscode-02.cm.cluster>
sidsingh-nvidia pushed a commit to sidsingh-nvidia/Megatron-LM that referenced this pull request Jul 13, 2026
Signed-off-by: Philip Petrakian <ppetrakian@nvidia.com>
chochowski pushed a commit to chochowski/Megatron-LM that referenced this pull request Jul 20, 2026
Signed-off-by: Philip Petrakian <ppetrakian@nvidia.com>
Signed-off-by: mchochowski <mchochowski@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved All necessary approvals have been made complexity: low

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants