DO NOT MERGE before learningequality/.github#88 — replace the seven automation callers with the central entry point - #6137
Conversation
Copy `automation-template.yml` from learningequality/.github verbatim to `.github/workflows/automation.yml`; routing now lives in the central reusable `automation.yml`, not in per-automation callers here. Trigger parity verified against the deleted callers: the only addition is `pull_request_review_comment: created`, no event or type is lost. The same seven secrets are passed. Merge only after learningequality/.github#88 — the copy calls `automation.yml@main`, which does not exist until that lands. Refs learningequality#6135 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
This adopts three automations the repo does not run today, not two. The third is
Net: merging as-is posts the holiday message on every pull request opened and every issue comment, year-round. The trigger-parity diff could not catch this. Class check — mapped all 13 registry entries against the 7 deleted callers. The callers cover 10 entries, not the 11 the issue states ( The copy is generated and must not be edited here, so the fix is Separately: the merge-order hold is the |
⛔ Merge order
Merge only after learningequality/.github#88 — the copy calls
learningequality/.github/.github/workflows/automation.yml@main, which does not exist until #88 lands.Summary
Every automation here fires through its own caller workflow carrying a hand-maintained
types:list,so enabling or disabling one means editing all eight consumer repos, and the lists have drifted. This
swaps the seven callers for one unedited copy of the generated
automation-template.yml, which routesevery event to the central reusable workflow. The repo gains two automations it did not run:
resolve-bot-pr-threads, via the newpull_request_review_comment: createdtrigger, andgood-first-issue-comment. Top-level permissions widen fromcontents: readtocontents: write+pull-requests: write, the union across automations, which the central workflow narrows back tocontents: readon every other job.Acceptance criteria not met
learningequality/.github@mainmainreturns 404. Copied verbatim from that branch — re-diff againstmainbefore merging.LE_BOT_APP_ID/LE_BOT_PRIVATE_KEYresolve in a rununassign-inactive-issuesfires Mondays 00:01 UTCcron: 1 0 * * 1.Reviewer guidance
gh api 'repos/learningequality/.github/contents/automation-template.yml?ref=consolidate-automation-entrypoint' --jq .content | base64 -d | diff - .github/workflows/automation.ymlon:blocks of the deleted callers (git show HEAD~1:.github/workflows/<file>) againstthe copy:
pull_request_review_comment: createdis added, nothing is removed, and the same sevensecret names get passed through.
The copy is generated upstream, so a defect in it is fixed in #88 rather than here.
References
automation-template.ymlatlearningequality/.github@consolidate-automation-entrypoint,sha256
83cd047c492b0a7d7949f12045a1e661003f3d11450f0663a92adfba06449f5dAI usage
Used Claude Code to fetch the template unedited, delete the seven callers, and diff trigger and secret
parity against the pre-deletion baseline. Verified with a byte-for-byte diff against upstream and
pre-commit run --all-files.🤖 Generated with Claude Code
@rtibblesbot's comments are generated by an LLM, and should be evaluated accordingly
How was this generated?
🟡 Waiting for feedback
Last updated: 2026-09-16 14:12 UTC