Skip to content

ci: Redline caller pinned to askalf/askalf@35a0ee3 (#62 + #63) - #120

Merged
askalf merged 2 commits into
masterfrom
ci/redline-pin-63
Sep 25, 2026
Merged

askalf merged 2 commits into
masterfrom
ci/redline-pin-63

Conversation

@askalf

@askalf askalf commented Sep 25, 2026

Copy link
Copy Markdown
Owner

What

.github/workflows/redline.yml calls askalf/askalf/.github/workflows/redline-review.yml at a fixed commit. This moves the pin from 63e007e to 35a0ee3, which carries askalf/askalf#62 and #63.

Why

At 18:24Z on 2026-09-25 eight repos ran their first Redline-in-CI job at once and five died with "no review submitted within 30 turns", while dario sat at its concurrency ceiling. The script at 63e007e nudged the model 24 times and paid 25 model calls per run for nothing, and logged nothing about what the replies held. At 35a0ee3 three replies without a tool call end the run, each such reply is logged with its text, an empty reply stays out of the history, and a reply with no content fails at once.

Nothing else changes

Same runner label, same guard, same permissions. The workflow is a pin bump only.

@github-actions github-actions Bot added github_actions Pull requests that update GitHub Actions code size/M 50-199 hand-written lines labels Sep 25, 2026
@askalf
askalf enabled auto-merge (squash) September 25, 2026 18:54
@github-actions github-actions Bot added size/XS Under 10 hand-written lines and removed size/M 50-199 hand-written lines labels Sep 25, 2026

@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.

Automated review from the Sprayberry Labs fleet code reviewer.
Reviewed by the gating lane (gating review).

Verdict: approve — no blocking issues found in this configuration-only pin update.

Checked the one-line reusable-workflow pin change, confirmed the target commit resolves in askalf/askalf, and reviewed the caller's unchanged same-repository guard, read-only permissions, and runner input. Required non-fleet checks are passing: actionlint, CodeQL analysis, docker-build, and unit-tests.

What's good: the workflow remains pinned to a full immutable commit SHA, while the inline source reference was updated consistently to the target change.

@askalf
askalf merged commit d84dde2 into master Sep 25, 2026
15 of 16 checks passed
@askalf
askalf deleted the ci/redline-pin-63 branch September 25, 2026 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code size/XS Under 10 hand-written lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants