Skip to content

Modify trigger labels for auto-merge in bulldozer.yml - #9

Merged
phuongfi91 merged 1 commit into
mainfrom
feat/phuongfi91/bulldozer-config
Aug 31, 2026
Merged

phuongfi91 merged 1 commit into
mainfrom
feat/phuongfi91/bulldozer-config

Conversation

@phuongfi91

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Phuong Nguyen <7949163+phuongfi91@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings August 31, 2026 08:05
@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@phuongfi91
phuongfi91 merged commit 345efd2 into main Aug 31, 2026
1 check passed
@phuongfi91
phuongfi91 deleted the feat/phuongfi91/bulldozer-config branch August 31, 2026 08:05

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The updated trigger criteria likely broadens auto-merge eligibility beyond the PR’s stated intent by removing branch scoping and making common labels sufficient to trigger merges.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Updates the repository’s Bulldozer auto-merge configuration to change which pull requests are eligible to be merged automatically.

Changes:

  • Expanded merge.trigger.labels to include dependencies and trunk in addition to merge when ready.
  • Commented out merge.trigger.branches, removing the explicit base-branch filter for triggering merges.
File summaries
File Description
bulldozer.yml Adjusts Bulldozer merge trigger criteria (labels and branch scoping).
Review details

Suppressed comments (1)

bulldozer.yml:28

  • Commenting out merge.trigger.branches removes the target-branch guardrail for merges and makes the trigger apply to any base branch as long as another trigger matches. The PR title suggests only label behavior should change; if this wasn't intentional, restore the branch list (and extend it explicitly if you need additional branches).
    # Pull requests targeting any of these branches are added to the trigger.
    # branches: ["dev", "main", "develop", "master"]
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread bulldozer.yml
Comment on lines 11 to +13
# Pull requests with any of these labels (case-insensitive) are added to
# the trigger.
labels: ["merge when ready"]
labels: ["dependencies", "trunk", "merge when ready"]
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.

2 participants