Repository navigation
ci: bump Redline to askalf/ci@9a559dc - #159
Conversation
sprayberry-redline
left a comment
There was a problem hiding this comment.
Verdict: request changes. Request changes because the pull request description says it updates drift-report action pins, but the diff changes only the two Redline reusable-workflow references and their inputs.
1. Blocking: PR description:1
Moves the pinned askalf/ci calls and their redline-ref inputs, and every drift-report action pin, to askalf/ci@9a559dc, the main commit that last changed the review, the fix or the report step (main 2026-10-17, askalf/ci#62). The scripts are fetched at that same commit. The pins are the only change.
This narrates updates to "every drift-report action pin," but neither changed workflow contains a drift-report action and the diff changes only the review and fix reusable-workflow pins plus their redline-ref values. The public description therefore claims a broader change than the patch makes.
Suggested fix:
Revise the description to state only the two reusable-workflow and redline-ref pin updates actually present, or include the claimed drift-report pin changes.
rule:reads-as-generated
Description updated at b5606a8
The pull request description is replaced. No file changed. |
sprayberry-redline
left a comment
There was a problem hiding this comment.
Verdict: approve. The reusable review and fix workflows remain pinned to the same full commit passed as redline-ref, so both workflow execution and script retrieval advance together.
Moves the two pinned askalf/ci reusable-workflow calls (redline-review.yml in redline.yml and redline-fix-run.yml in redline-fix.yml) and their redline-ref inputs to askalf/ci@9a559dc, the main commit that last changed the review, the fix or the report step (main 2026-10-17, askalf/ci#62). The scripts are fetched at that same commit. The pins are the only change.