Skip to content

Redline pin bump: add askalf/redline-rubric to CALLERS - #213

Merged
askalf merged 1 commit into
mainfrom
rubric-pin-bump
Nov 12, 2026
Merged

askalf merged 1 commit into
mainfrom
rubric-pin-bump

Conversation

@askalf

@askalf askalf commented Nov 12, 2026

Copy link
Copy Markdown
Owner

askalf/redline-rubric is reviewed on the runner host like the other callers (pitlane-box#150, platform#2340), and the askalf-pin-bump App is now installed on it. This adds it to CALLERS in redline-pin-bump.yml, so each pin wave moves its review, public-text and post callers.

  • scripts/redline/review.test.mjs: the expected callers list gains it (twenty).
  • scripts/redline/pin-bump.test.mjs: unchanged. Its limit check now reads 20 callers at 70 s each with 1.25x room, 1750 s of the 1800 s limit, and passes.
  • CHANGELOG entry.

No prompt text and no rubric text are added.

@sprayberry-redline sprayberry-redline left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: approve. Adding askalf/redline-rubric to CALLERS includes it in the existing pin-bump loop, and the updated assertion checks the full twenty-repository list. The changelog matches the change, and the caller count remains within the existing timeout budget.

@askalf
askalf merged commit 00c9573 into main Nov 12, 2026
13 checks passed
@askalf
askalf deleted the rubric-pin-bump branch November 12, 2026 16:50
askalf pushed a commit to askalf/pitlane that referenced this pull request Nov 12, 2026
Moves this repository's askalf/ci pins (its reusable workflow calls,
their redline-ref inputs and any action pins) to
askalf/ci@00c9573, the main commit that
last changed the review, the fix or an action (main 2026-11-12,
askalf/ci#213). The scripts are fetched at that same commit. The pins
are the only change.

---------

Co-authored-by: askalf-pin-bump[bot] <352119504+askalf-pin-bump[bot]@users.noreply.github.com>
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