Skip to content

ci: remove automerge job from pull-request-target#5753

Merged
max-sixty merged 1 commit intomainfrom
ci/remove-automerge
Mar 28, 2026
Merged

ci: remove automerge job from pull-request-target#5753
max-sixty merged 1 commit intomainfrom
ci/remove-automerge

Conversation

@prql-bot
Copy link
Copy Markdown
Collaborator

Summary

  • Remove the automerge job from pull-request-target.yaml
  • Branch protection rules now gate merges, so bot PRs (from prql-bot, dependabot[bot], pre-commit-ci[bot]) no longer need to be auto-merged — they'll go through the same review flow as all other PRs

Context: #5752 was auto-merged by this job before a maintainer could review it. With merge protection in place, the automerge job is redundant.

Test plan

  • Verified automerge is not referenced as a required check elsewhere in the repo
  • CI passes on this PR (remaining jobs in pull-request-target.yaml — PR title validation and backport — are unchanged)

🤖 Generated with Claude Code

Branch protection rules now provide merge gating, making the automerge
job redundant. Removing it ensures bot PRs (from prql-bot, dependabot,
pre-commit-ci) go through the same review process as all other PRs.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@prql-bot prql-bot enabled auto-merge (squash) March 28, 2026 21:23
@max-sixty max-sixty disabled auto-merge March 28, 2026 21:36
@max-sixty max-sixty merged commit 4f78c92 into main Mar 28, 2026
41 checks passed
@max-sixty max-sixty deleted the ci/remove-automerge branch March 28, 2026 21:36
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