Skip to content

ci: pin the Redline review to askalf/askalf main at #72 and pass redline-ref - #96

Merged
askalf merged 1 commit into
mainfrom
ci/redline-pin-f97c635
Sep 26, 2026
Merged

askalf merged 1 commit into
mainfrom
ci/redline-pin-f97c635

Conversation

@askalf

@askalf askalf commented Sep 26, 2026

Copy link
Copy Markdown
Owner

This moves the redline-review.yml pin to askalf/askalf main at f97c635 (askalf/askalf#72) and passes the same sha as redline-ref. The review script is now fetched at that pinned sha, which the called workflow checks is a commit on askalf/askalf main, instead of always running whatever is on main. The uses sha and redline-ref must always name the same commit.

…ine-ref

The review script is now fetched at the pinned sha, which the called workflow checks is on askalf/askalf main. The uses sha and redline-ref must always match.

@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. Bumps the reusable Redline review workflow pin from 58a8e5f to f97c635 and passes that same full-length sha as the new redline-ref input, so the review script is fetched at the reviewed commit rather than whatever is on main. The uses sha and the redline-ref value are identical, the pin stays a full 40-character commit sha, and the triggers, read-only permissions and same-repo/non-draft gate are untouched.

@askalf
askalf enabled auto-merge (squash) September 26, 2026 14:02
@github-actions github-actions Bot added github_actions Pull requests that update GitHub Actions code size/XS Under 10 hand-written lines labels Sep 26, 2026
@askalf
askalf merged commit e9d3664 into main Sep 26, 2026
11 checks passed
@askalf
askalf deleted the ci/redline-pin-f97c635 branch September 26, 2026 14:04
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