Skip to content

revert(deps): pin go back to 1.26.6, undo #650 (RIG-3303) - #901

Closed
rigel-mintaka wants to merge 1 commit into
mainfrom
compass-repo/rig-3303-revert-go-1271
Closed

revert(deps): pin go back to 1.26.6, undo #650 (RIG-3303)#901
rigel-mintaka wants to merge 1 commit into
mainfrom
compass-repo/rig-3303-revert-go-1271

Conversation

@rigel-mintaka

Copy link
Copy Markdown
Contributor

Renovate #650 bumped the go compiler 1.26.6->1.27.1, but nilaway (the gating nil-flow analyzer in the pinned nixpkgs devenv shell) is a go1.26-built binary that cannot parse the go1.27 stdlib. It fails with file requires newer Go version go1.27 (application built with go1.26) and unknown field rfd in struct literal of type splicePipe, reddening compass-go:nilaway -> moon (go) -> rollup (the required check) and gating the entire merge queue since 2026-09-05T17:27Z.

Pure revert: restores tools/toolchain/versions/go.nix to 1.26.6 and devenv.lock go-overlay rev to the pre-bump value, byte-identical to the pre-#650 tree. go.mod floor is go 1.25.0, unaffected. The durable regroup (pin a go1.27-built nilaway + group go+nilaway in Renovate, then re-land) is tracked as a dependencies-lane follow-up.

Spec-impact: none. Refs RIG-3303. Refs #650.

Co-authored-by: Matt Wilkinson matt@rigel.build

Renovate #650 bumped the go compiler 1.26.6->1.27.1, but nilaway (the gating nil-flow analyzer in the pinned nixpkgs devenv shell) is a go1.26-built binary that cannot parse the go1.27 stdlib. It fails with `file requires newer Go version go1.27 (application built with go1.26)` and `unknown field rfd in struct literal of type splicePipe`, reddening `compass-go:nilaway` -> `moon (go)` -> `rollup` (the required check) and gating the entire merge queue since 2026-09-05T17:27Z.

Pure revert: restores `tools/toolchain/versions/go.nix` to 1.26.6 and `devenv.lock` go-overlay rev to the pre-bump value, byte-identical to the pre-#650 tree. go.mod floor is `go 1.25.0`, unaffected. The durable regroup (pin a go1.27-built nilaway + group go+nilaway in Renovate, then re-land) is tracked as a dependencies-lane follow-up.

Spec-impact: none. Refs RIG-3303. Refs #650.

Co-authored-by: Matt Wilkinson <matt@rigel.build>
@linear-code

linear-code Bot commented Sep 5, 2026

Copy link
Copy Markdown

RIG-3303

@trunk-io

trunk-io Bot commented Sep 5, 2026

Copy link
Copy Markdown

Merging to main in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown

Compass engineering docs preview: https://compass-repo-rig-3303-revert.compass-eng-docs.pages.dev

Deployed from compass-repo/rig-3303-revert-go-1271 at dc3640d.

@rigel-mintaka

Copy link
Copy Markdown
Contributor Author

Superseded by #904 (RIG-3303), merged to main at ab56c27. #904 was the fix-forward Matt ruled for (build the Go analysis battery with the go1.27 toolchain); this pure-revert-to-go1.26.6 was the break-glass fallback only and is no longer needed. Closing.

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