Skip to content

ci(scrape): enable daily staged cron + document ledger-to-main bypass requirement#21

Merged
KiranPranay merged 1 commit into
mainfrom
enable-cron
Jul 10, 2026
Merged

ci(scrape): enable daily staged cron + document ledger-to-main bypass requirement#21
KiranPranay merged 1 commit into
mainfrom
enable-cron

Conversation

@KiranPranay

Copy link
Copy Markdown
Member

Your directive #2 (cron), plus the bypass documentation for #1.

Cron enabled — daily 06:00 IST staged run; LAUNCH_MODE stays staged (opens/updates the data-review PR only, no publish). Rehearsal week begins; counts toward the 2/day cap.

Ledger-to-main (#1) is blocked, honestly reported: github-actions[bot] cannot be added to the ruleset bypass — GitHub returns 422 ("the Actions integration must be part of the ruleset source or owner org"). So the bypass path needs SCRAPE_BOT_TOKEN to be an admin PAT or a repo-installed GitHub App added to the main-protection bypass (push-only). Documented in CONTRIBUTING §Operations; LEDGER_TO_MAIN stays off until then so runs don't fail on the push. (The ruleset is unchanged — my add attempts 422'd and applied nothing.)

…-to-main bypass reality

Enable the daily schedule (00:30 UTC = 06:00 IST). LAUNCH_MODE stays 'staged', so
each cron run only opens/updates the data-review PR — nothing publishes without a
human merge. Begins the operational rehearsal week; counts toward the 2/day cap.

CONTRIBUTING §Operations: document that github-actions[bot] CANNOT be granted
ruleset bypass (GitHub 422), so LEDGER_TO_MAIN requires SCRAPE_BOT_TOKEN to be an
admin PAT or a repo-installed GitHub App added to the main-protection bypass
(push-only; PR + checks stay enforced for everyone else). Kept LEDGER_TO_MAIN off
until such a token exists, so runs don't fail on the blocked push.
@KiranPranay KiranPranay merged commit 2a3d83f into main Jul 10, 2026
3 checks passed
@KiranPranay KiranPranay deleted the enable-cron branch July 10, 2026 18:32
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.

1 participant