Skip to content

Conversation

@silviogutierrez
Copy link
Contributor

@silviogutierrez silviogutierrez commented Feb 1, 2026

Summary

  • Add matrix strategy to test against Go 1.21, 1.22, and 1.23 using nix
  • Update shell.nix to accept a goPackage argument (defaults to go for local dev)
  • CI passes --arg goPackage "go_1_xx" to test each version

Test plan

  • Verify CI runs for all three Go versions
  • Confirm all tests pass on each version
  • Verify local nix-shell still works with default Go

🤖 Generated with Claude Code

@silviogutierrez silviogutierrez force-pushed the ci-multi-go-versions branch 6 times, most recently from 3f7439c to 2c24dee Compare February 1, 2026 23:09
Test against Go 1.17 through 1.22 using a matrix strategy with nix.
The shell.nix accepts a goPackage argument and maps versions to
different nixpkgs pins:
- pkgsCurrent (8c5066250910): go_1_21, go_1_22
- pkgs2305 (nixos-23.05): go_1_18, go_1_19, go_1_20
- pkgs2205 (nixos-22.05): go_1_17

Adds a "ci" rollup job that can be used as a single required check
in branch protection rules.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copy link
Owner

@ajg ajg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good; can just update the README please and go.mod?

- Update README to reflect Go 1.17 minimum requirement
- Lower go.mod version to 1.17 to match CI matrix

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@ajg ajg merged commit c6078f1 into ajg:master Feb 1, 2026
7 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.

2 participants