Skip to content
This repository was archived by the owner on Jan 17, 2026. It is now read-only.

fix: Allow automerge for branch patterns regardless of actor#84

Merged
duyluann merged 1 commit into
mainfrom
fix-automerge-branch-pattern
Jan 12, 2026
Merged

fix: Allow automerge for branch patterns regardless of actor#84
duyluann merged 1 commit into
mainfrom
fix-automerge-branch-pattern

Conversation

@duyluann

@duyluann duyluann commented Jan 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • Remove actor restrictions for branch-based automerge conditions
  • Allow auto-merge for PRs with branch names matching update-license-* or update-pre-commit-hooks regardless of who creates them
  • Keep dependabot auto-merge support

Test plan

  • Verify that PRs with matching branch patterns trigger automerge workflow
  • Confirm existing dependabot functionality remains intact

Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Simplified the automerge workflow configuration for improved maintainability.

Note: This is an internal infrastructure update with no end-user visible changes.

✏️ Tip: You can customize this high-level summary in your review settings.

Remove actor restrictions for branch-based automerge conditions. The workflow now auto-merges PRs based on branch name patterns (update-license-*, update-pre-commit-hooks) regardless of who created them, while still supporting dependabot PRs.
@coderabbitai

coderabbitai Bot commented Jan 12, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

A GitHub workflow automerge conditional was refactored from a nested grouped boolean expression into a flattened OR structure. The condition now directly evaluates PR head branches and actor identity without parenthetical grouping while maintaining equivalent logic.

Changes

Cohort / File(s) Summary
Automerge Workflow Configuration
.github/workflows/automerge.yml
Simplified the if clause condition by flattening nested boolean grouping. Changed from grouped expressions checking github-actions[bot] with specific PR branches OR dependabot[bot] to a direct OR evaluation of multiple conditions without intermediate grouping.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Poem

🐰 A conditional once nested deep,
Now flattens out, so clean and neat!
OR statements aligned just right,
The workflow flows with cleaner sight! ✨


📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 69017c3 and d7b5025.

📒 Files selected for processing (1)
  • .github/workflows/automerge.yml

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.

@duyluann duyluann merged commit 5ee45ef into main Jan 12, 2026
9 of 10 checks passed
@duyluann duyluann deleted the fix-automerge-branch-pattern branch January 12, 2026 06:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant