Skip to content

fix(ci): explicitly select Go 1.27.3 in setup-go - #73

Merged
steipete merged 1 commit into
mainfrom
security/go1273-ci
Oct 21, 2026
Merged

steipete merged 1 commit into
mainfrom
security/go1273-ci

Conversation

@steipete

Copy link
Copy Markdown
Contributor

Explicitly select Go 1.27.3 in every setup-go invocation, preserving the Go 1.27.1 source minimum and existing source toolchain declaration.

Follow-up to #72: go-version-file makes setup-go select the language minimum before consulting the toolchain directive. The explicit version prevents this older setup selection in CI, release, benchmark, and results workflows. All six published v0.2.1 binaries already independently verify as Go 1.27.3 through source toolchain switching; no replacement release is needed.

Validation: actionlint passes; independent P0–P3 autoreview found no actionable issues. Exact-head CI verifies both Linux and macOS packaging.

@steipete
steipete requested review from a team and Patrick-Erichsen as code owners October 21, 2026 03:45
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-21T03:47:14.144843Z eb135fc PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@clawsweeper

clawsweeper Bot commented Oct 21, 2026

Copy link
Copy Markdown

🦞👀
ClawSweeper picked this up.

Pull request received. I will update this pull request when review starts.

@steipete
steipete merged commit 4d2f57f into main Oct 21, 2026
8 of 9 checks passed
@steipete
steipete deleted the security/go1273-ci branch October 21, 2026 03:48
@steipete

Copy link
Copy Markdown
Contributor Author

Merged after exact-head CI passed. Every setup-go invocation now explicitly selects Go 1.27.3 while preserving the language minimum and module toolchain declaration. The pinned action gives this explicit input precedence over go-version-file. Actionlint and the applicable workflow contract tests passed; independent P0–P3 autoreview was clean. The transient ClawSweeper intake timeout was diagnosed from the failed job log and the failed-job retry 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.

1 participant