Skip to content

Group automation jobs by event and move the PR spreadsheet to a daily cron - #101

Open
rtibbles wants to merge 2 commits into
mainfrom
group-automation-jobs-by-event
Open

rtibbles wants to merge 2 commits into
mainfrom
group-automation-jobs-by-event

Conversation

@rtibbles

Copy link
Copy Markdown
Member

Summary

  • Automation workflow was triggering a lot of check runs on every PR
  • Reduce this by:
    • Nesting dispatches of the same events under a job - if that event hasn't happened, fewer triggers run
    • Switching the spreadsheet update into a daily cron

References

First seen: learningequality/kolibri#15376. Branch test runs: learningequality/test-actions#98, learningequality/test-actions#99.

Reviewer guidance

  1. On the test-actions PR and issue in References, compare each event's "Automation" and "Automation (branch test)" runs — the same jobs run, with fewer checks.
  2. Actions > "Update community pull requests spreadsheet" > Run workflow on group-automation-jobs-by-event with dry_run ticked — the log lists contributor PRs updated in the last two days and writes nothing.
  • Does daily work for updating the spreadsheet? I assume we don't need to keep track of it up to the minute.
  • The spreadsheet now covers every public org repo except test-actions, not only repos with the automation caller.
  • automation.yml keeps declaring the three spreadsheet secrets as retired_secrets, since consumers forward them until their sync PR merges.

AI usage

Used Claude Code to implement the grouped generator output and the spreadsheet cron, and to exercise the grouping in test-actions. Verified with the node test suite, prek, branch runs in test-actions, and a local dry run of the spreadsheet query.

🤖 Generated with Claude Code

https://claude.ai/code/session_01R7BDHRZZKPZdFbjvLLke2h

rtibbles and others added 2 commits September 25, 2026 16:08
Every job in an Automation run posts a check on the PR, even when skipped,
so each PR event added ~18 checks. Automations sharing trigger events now
sit behind one reusable group workflow, which posts a single check when
skipped.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01R7BDHRZZKPZdFbjvLLke2h
update-pr-spreadsheet was the only automation on six pull_request_target
actions, so every PR edit, assignment or close started an Automation run
in every consumer repo. It now runs once a day here, upserting public
contributor PRs across the org updated in the last two days.

- Template drops those six actions and the three spreadsheet secrets
- automation.yml still accepts the secrets (retired_secrets) until
  every consumer has synced, since forwarding an undeclared secret fails
- Scope widens from consumer repos to all public org repos

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01R7BDHRZZKPZdFbjvLLke2h
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