Skip to content

chore: update Go to 1.27.1 and golangci-lint to v2.14 - #183

Merged
pjcdawkins merged 2 commits into
mainfrom
claude/upgrade-go-lint-actions-b5d302
Sep 25, 2026
Merged

pjcdawkins merged 2 commits into
mainfrom
claude/upgrade-go-lint-actions-b5d302

Conversation

@pjcdawkins

Copy link
Copy Markdown
Contributor
  • Update Go from 1.26.2 to 1.27.1. setup-go installs exactly the version in go.mod, so releases were still built with 1.26.2.
  • Remove GOEXPERIMENT=jsonv2: Go 1.27 enables it by default, so released binaries now use the v2-backed encoding/json that tests already ran against.
  • Update golangci-lint to v2.14 in GitHub Actions and GitLab CI, and the GitLab Go image to cimg/go:1.27. Remove a //nolint:gosec that is no longer needed.
  • Remove internal/certs. Since Go 1.27, crypto/x509 honors SSL_CERT_FILE and SSL_CERT_DIR on Windows and macOS too, and for all HTTP transports. An unreadable SSL_CERT_FILE now fails verification instead of warning and falling back to system certificates, matching the existing Linux behavior.

GitHub Actions were already on their latest major versions.

🤖 Generated with Claude Code

pjcdawkins and others added 2 commits September 25, 2026 23:23
- Bump the go directive to 1.27.1; CI picks it up via go-version-file.
- Drop GOEXPERIMENT=jsonv2: Go 1.27 enables the jsonv2 experiment by
  default.
- Pin golangci-lint v2.14 in GitHub Actions, and update the GitLab CI
  images (cimg/go:1.27, golangci-lint:v2.14).
- Remove a //nolint:gosec directive that gosec 2.29 no longer needs.

GitHub Actions were already on their latest major versions.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Since Go 1.27, crypto/x509 honors SSL_CERT_FILE and SSL_CERT_DIR on
Windows and macOS too (GODEBUG x509sslcertoverrideplatform), so the Go
part of the CLI trusts the same certificates as the legacy PHP part
without a workaround. This also covers HTTP transports other than the
default one.

An unreadable SSL_CERT_FILE now fails verification instead of printing
a warning and falling back to the system certificates, as it already
did on Linux.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@pjcdawkins
pjcdawkins merged commit 837566e into main Sep 25, 2026
6 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.

1 participant