Skip to content

fix: disable auto merger and branch checker in ops-bot configuration - #380

Merged
AjayThorve merged 1 commit into
NVIDIA:mainfrom
AjayThorve:ci/disable-auto-merger
Oct 10, 2026
Merged

AjayThorve merged 1 commit into
NVIDIA:mainfrom
AjayThorve:ci/disable-auto-merger

Conversation

@AjayThorve

@AjayThorve AjayThorve commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator

Overview

Keep only ops-bot forward merging enabled.

  • I confirm this contribution is my own work, or I have the right to submit it under this project's license.
  • I searched existing issues and open pull requests, and this does not duplicate existing work.

Details

Set auto_merger and branch_checker to false in .github/ops-bot.yaml. Keep label_checker and release_drafter set to false, and retain forward_merger: true.

Validation: staged repository pre-commit hooks passed, including YAML validation, file hygiene, copyright checks, and documentation link checking. git diff --cached --check passed. Language test suites were not run because this change only updates ops-bot configuration. No public API changes.

Where should the reviewer start?

Review the two changed values in .github/ops-bot.yaml and confirm forward_merger is the only enabled option.

Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)

  • Relates to: none

Summary by CodeRabbit

  • Configuration
    • Automatic merging and branch checking are now disabled. Other automation settings remain unchanged. This may affect whether changes are merged automatically and whether branches are checked through these automated processes.

@AjayThorve
AjayThorve requested a review from a team as a code owner October 9, 2026 18:12
@coderabbitai

coderabbitai Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: NVIDIA/NeMo-Fabric/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Enterprise
  • Run ID: 23221260-177c-4207-ae61-e3794b8bbf78

📥 Commits

Reviewing files that changed from the base of the PR and between c359c4d and fb4ebe5.


📒 Files selected for processing (1)
  • .github/ops-bot.yaml

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 10 remain after this review.


📜 Recent review details
⏰ Context from checks skipped due to timeout. (27)
  • GitHub Check: Test adapters (Node 22.19.0)
  • GitHub Check: Test (Node 20.18.3)
  • GitHub Check: Test adapters (Node 24)
  • GitHub Check: Test (Node 24)
  • GitHub Check: Test (x86_64)
  • GitHub Check: Pre-commit
  • GitHub Check: Test (Python 3.11, windows-amd64)
  • GitHub Check: Test (arm64)
  • GitHub Check: Test (Python 3.13, linux-amd64)
  • GitHub Check: Cline E2E
  • GitHub Check: Test (Python 3.13, windows-amd64)
  • GitHub Check: Test (Python 3.12, windows-amd64)
  • GitHub Check: Test (Python 3.14, windows-amd64)
  • GitHub Check: Test (Python 3.13, linux-arm64)
  • GitHub Check: Test (Python 3.11, macos-arm64)
  • GitHub Check: Test (Python 3.14, macos-arm64)
  • GitHub Check: Test (Python 3.13, macos-arm64)
  • GitHub Check: Test (Python 3.11, linux-amd64)
  • GitHub Check: Test (Python 3.14, linux-amd64)
  • GitHub Check: Test (Python 3.12, macos-arm64)
  • GitHub Check: Test (Python 3.12, linux-arm64)
  • GitHub Check: OpenCode E2E
  • GitHub Check: Test (Python 3.14, linux-arm64)
  • GitHub Check: Test (Python 3.11, linux-arm64)
  • GitHub Check: Test (Python 3.12, linux-amd64)
  • GitHub Check: Hermes adapter (upstream Relay 0.9, Python 3.14)
  • GitHub Check: Qwen Code E2E

🔇 Additional comments (1)
.github/ops-bot.yaml (1)

7-8: LGTM!





Walkthrough

The ops bot configuration now disables auto_merger and branch_checker. Other feature settings remain unchanged.

Changes

Ops bot settings

Layer / File(s) Summary
Disable selected features
.github/ops-bot.yaml
auto_merger and branch_checker are set to false. Other feature settings are unchanged.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to fb4eb

The configuration leaves the documented forward-merger option enabled and disables the other selected features. Available evidence establishes no concrete merge-blocking issue.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check Passed The title follows Conventional Commits format. It uses the allowed lowercase type fix, uses an imperative summary, stays within 72 characters, and has no trailing period.
Description check Passed The description includes the required Overview, reviewer-start, Related Issues, and contribution checklist sections. It also explains the configuration changes and validation results.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.


✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR


  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@AjayThorve
AjayThorve merged commit 916b229 into NVIDIA:main Oct 10, 2026
43 checks passed

This branch was successfully deployed

1 active deployment
fern — fb4ebe58 Deployed Oct 10, 2026 by AjayThorve via Clean up docs preview #1990
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant