Skip to content

scorecard: check out with actions/checkout so results publish - #195

Merged
askalf merged 1 commit into
mainfrom
scorecard-publish-checkout
Nov 1, 2026
Merged

askalf merged 1 commit into
mainfrom
scorecard-publish-checkout

Conversation

@askalf

@askalf askalf commented Nov 1, 2026

Copy link
Copy Markdown
Owner

The Scorecard workflow checks out with actions/checkout again instead of askalf/ci/actions/checkout-with-retry.

scorecard.dev accepts published results only from a workflow made of steps it recognises. Since the checkout moved to the retry wrapper, every run here has passed but its upload was turned away, so the badge and the scorecard.dev API still show the last score from before that change.

This job runs on a hosted runner, where the retry is not needed. The step now matches askalf/ci's own Scorecard workflow: the same pinned actions/checkout v7.0.1 with persist-credentials: false, and a comment that says why. Nothing else in the file changes.

Proof after merge: the next Scorecard run on the default branch publishes, and https://api.scorecard.dev/projects/github.com/askalf/<repo> shows that run's date.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

Deploying amnesia-site with  Cloudflare Pages  Cloudflare Pages

Latest commit: df17983
Status: ✅  Deploy successful!
Preview URL: https://32463ad7.amnesia-site.pages.dev
Branch Preview URL: https://scorecard-publish-checkout.amnesia-site.pages.dev

View logs

@github-actions github-actions Bot added github_actions Pull requests that update GitHub Actions code size/XS Under 10 hand-written lines labels Nov 1, 2026
@askalf
askalf enabled auto-merge (squash) November 1, 2026 20:15

@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. The workflow now uses a directly pinned actions/checkout step, matching the checkout form expected for Scorecard publication. The change preserves persist-credentials: false and leaves the analysis and upload steps unchanged.

@askalf
askalf merged commit 544e689 into main Nov 1, 2026
15 checks passed
@askalf
askalf deleted the scorecard-publish-checkout branch November 1, 2026 20:22
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