Skip to content

Revert merge_group CI triggers: merge queue unavailable on this repo - #194

Merged
BigSimmo merged 1 commit into
mainfrom
claude/remove-dead-merge-group-triggers
Jul 2, 2026
Merged

Revert merge_group CI triggers: merge queue unavailable on this repo#194
BigSimmo merged 1 commit into
mainfrom
claude/remove-dead-merge-group-triggers

Conversation

@BigSimmo

Copy link
Copy Markdown
Owner

Summary

Reverts the merge_group workflow triggers added in #183. A GitHub merge queue cannot be enabled on this repository — it's private and owned by a personal account, and GitHub gates merge queue to public repos and organization-owned repos on Team/Enterprise (the API rejects the merge_queue ruleset rule with a validation error). That makes the merge_group triggers dead config no queue will ever fire, so this restores ci.yml and secret-scan.yml to their pre-#183 state (also dropping the now-unnecessary gitleaks merge_group guard).

Paired change (already applied, not in this diff): the strict up-to-date ruleset policy (strict_required_status_checks_policy) has been reverted to OFF, back to the velocity-friendly setup that suits this high-churn multi-agent repo. Required checks — verify, Gitleaks, ui-smoke — remain enforced.

Verification

  • Both workflows validated as YAML; merge_group removed from on
  • Gitleaks step restored to a single unconditional Scan for secrets
  • verify:ui — N/A: CI-config only

Clinical Governance Preflight

N/A — CI workflow triggers only; no ingestion, answer-generation, search, source, access, privacy, or clinical-output change.

Notes

🤖 Generated with Claude Code

Merge queue can't be enabled on this repo (private + user-owned; GitHub
gates the feature to public repos and org-owned repos on Team/Enterprise),
so the merge_group triggers added in #183 are dead config that no queue
will ever fire. Restore ci.yml and secret-scan.yml to their pre-#183
state, which also drops the now-unnecessary gitleaks merge_group guard.
The strict up-to-date ruleset policy has been reverted to OFF separately,
returning to the velocity-friendly setup that suits this high-churn repo.
Required checks (verify, Gitleaks, ui-smoke) remain enforced.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@BigSimmo
BigSimmo merged commit b447e8d into mainJul 2, 2026
4 checks passed
@BigSimmo
BigSimmo deleted the claude/remove-dead-merge-group-triggers branch July 2, 2026 15:02
BigSimmo pushed a commit that referenced this pull request Aug 14, 2026
…merged
#215, #210, and #293 were queued as `done` in this same PR, each citing a PR
(#1949, #1953, #1962) that turned out still to be open, not merged — verified
by checking whether each PR's actual file change is present on origin/main
(none are). Reconciling the original `done` requests would have closed these
rows while their fixes exist only on unmerged branches, one of which (#1949)
currently has failing required CI.
Converts all three to `update` requests documenting the correction and the
real current state, so reconciling this PR now cannot close a row before its
fix has actually landed. Leaves #98, #189, and #194 as `done` unchanged —
those cite PR #1950 and PR #1947, both confirmed merged into main.
Sign up for freeto 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

@BigSimmo