Skip to content

feat: reconciler enables allow_auto_merge (queue-usable on all + new repos)#31

Merged
Aswinmcw merged 1 commit into
mainfrom
feat/reconciler-enable-automerge
Jul 15, 2026
Merged

feat: reconciler enables allow_auto_merge (queue-usable on all + new repos)#31
Aswinmcw merged 1 commit into
mainfrom
feat/reconciler-enable-automerge

Conversation

@Aswinmcw

Copy link
Copy Markdown
Member

Why

Merge queues require allow_auto_merge enabled to be usable. Only the 6 originally-tuned repos had it; the 13 I added queues to (PR #24) had it off, so their queue couldn't be driven by --auto / automation. Backfilled all 19 by API.

This PR — make it durable + apply to new repos

Adds ensure_repo_settings(repo) — an idempotent helper that enables allow_auto_merge on every managed repo and any new repo the reconciler provisions (reads current state first; only PATCHes if off; dry-run aware). Result folded into the run report note.

Important (not a behavior change you'll notice)

Enabling the setting does not auto-merge anything. It only unlocks the capability. Your own PRs still sit until you click Merge when ready; unattended merge happens only if a PR is explicitly armed (e.g. the update-readme bot). Verified live: with the setting on, a PR merges unattended only after being armed; the manual queue button and a plain approved PR both wait for you.

Merge queues need allow_auto_merge enabled to be usable (the --auto/enable-
auto-merge path and the queue button). It was on only for the 6 hand-tuned
repos; the 13 I added queues to had it off, so their queue was effectively
unusable via automation. Backfilled all 19; this makes it durable: an
idempotent ensure_repo_settings() flips allow_auto_merge=true for every managed
repo and any NEW repo the reconciler provisions. Enabling the setting does not
merge anything on its own — it only permits arming auto-merge / using the queue;
your own PRs still wait for a click.
@Aswinmcw Aswinmcw requested review from a team and Aswin-coder as code owners July 15, 2026 11:23

@Aswincloud-Bot Aswincloud-Bot 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.

Auto-approved: @Aswinmcw is a member of @Aswincloud/admins.

@Aswinmcw Aswinmcw added this pull request to the merge queue Jul 15, 2026
Merged via the queue into main with commit f93f9ca Jul 15, 2026
2 checks passed
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