Skip to content

ci: actionlint, lock down checkout, race tests on three OSes - #8

Merged
xeaser merged 2 commits into
mainfrom
ci/broader-overhaul
Aug 21, 2026
Merged

xeaser merged 2 commits into
mainfrom
ci/broader-overhaul

Conversation

@xeaser

@xeaser xeaser commented Aug 21, 2026

Copy link
Copy Markdown
Owner

What

Broader CI slice (squad-h7s.4): lint workflows, stop jobs from pushing with the default checkout token, and run race tests on all three OSes. One PR.

How to test

  • actionlint v1.7.12 clean on ci.yml and release.yml locally
  • PR ci check green (lint, test matrix with -race on Ubuntu/Windows/macOS, vuln, build, actionlint)
  • Docs updated if a user-visible command or flag changed (CONTRIBUTING CI sentence only)

Checklist

  • Commits are signed (required on the default branch)
  • No secrets, no .opencode/node_modules
  • Does not add Copilot, Ink shell, Aspire, or npm install

Changes

  • New actionlint job, pinned 1.7.12 (download script from the v1.7.12 tag)
  • persist-credentials: false on every actions/checkout in ci.yml and release.yml
  • Job-level contents: read on jobs that check out; gate job permissions: {}
  • test uses the same OS matrix as build (fail-fast: false) with go test -race -count=1
  • CONTRIBUTING: CI tests (not only builds) on Linux/Windows/macOS

Out of this PR: SHA-pin actions, Dependabot, tag ruleset, zizmor. If Windows -race flakes, follow-up drops -race on Windows only.

xeaser added 2 commits August 21, 2026 12:49
Add a pinned actionlint 1.7.12 job, persist-credentials: false on
every checkout, explicit job-level contents: read, and go test
-race on Ubuntu, Windows, and macOS. Release checkout cannot push.

Beads: squad-h7s.4
ApplyTheme office then none failed on Windows because the H1 rewrite
dropped CR. Restore keeps the original line ending.

Beads: squad-h7s.4
@xeaser
xeaser merged commit 8deef3d into main Aug 21, 2026
10 checks passed
@xeaser
xeaser deleted the ci/broader-overhaul branch August 21, 2026 08:09
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.

1 participant