Skip to content

runner-preflight drops the PR-creation check and its… - #171

Merged
brunogbv merged 1 commit into
mainfrom
chore/149-runner-preflight-steward
Oct 17, 2026
Merged

brunogbv merged 1 commit into
mainfrom
chore/149-runner-preflight-steward

Conversation

@brunogbv

Copy link
Copy Markdown
Collaborator

The check probed whether the host token could open PRs and, for Scala Steward, recommended a
STEWARD_GH_TOKEN PAT. Since MIP-0086 Steward mints a marola-pr-bot token, runs on ubuntu-latest,
and marola-dev/marola#765 deleted that secret; only marola-ml's GPU publish may use the
self-hosted runner. Nothing on the runner needs a token that opens PRs, so the probe, its verdict
function, the host-token helper and their four self-test cases go.

Closes #149

Tested: devkit-quality.sh (just quality's steps without Nix) rc=0; runner-preflight.sh --self-test ok; runner-preflight.sh on a non-runner host with gh signed out prints no Steward or PAT line; git grep STEWARD_GH_TOKEN -- scripts docs README.md prints nothing
Co-Authored-By: Claude noreply@anthropic.com

@github-actions github-actions Bot changed the title runner-preflight drops the PR-creation check and its STEWARD_GH_TOKEN advice runner-preflight drops the PR-creation check and its… Oct 17, 2026
… advice

The check probed whether the host token could open PRs and, for Scala Steward, recommended a
STEWARD_GH_TOKEN PAT. Since MIP-0086 Steward mints a marola-pr-bot token, runs on ubuntu-latest,
and marola-dev/marola#765 deleted that secret; only marola-ml's GPU publish may use the
self-hosted runner. Nothing on the runner needs a token that opens PRs, so the probe, its verdict
function, the host-token helper and their four self-test cases go.

Closes #149

Tested: devkit-quality.sh (just quality's steps without Nix) rc=0; runner-preflight.sh --self-test ok; runner-preflight.sh on a non-runner host with gh signed out prints no Steward or PAT line; git grep STEWARD_GH_TOKEN -- scripts docs README.md prints nothing
Co-Authored-By: Claude <noreply@anthropic.com>
@brunogbv
brunogbv force-pushed the chore/149-runner-preflight-steward branch from 2b84612 to fb5002b Compare October 17, 2026 11:09
@brunogbv
brunogbv merged commit 26fd98f into main Oct 17, 2026
8 checks passed
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.

runner-preflight stops advising the STEWARD_GH_TOKEN PAT; Scala Steward runs as marola-pr-bot

1 participant