Skip to content

gomod: Update Go toolchain to 1.25.3 and remove 1.24 compatibility#55

Merged
echarrod merged 3 commits intomainfrom
update-go-1.25.3
Nov 7, 2025
Merged

gomod: Update Go toolchain to 1.25.3 and remove 1.24 compatibility#55
echarrod merged 3 commits intomainfrom
update-go-1.25.3

Conversation

@echarrod
Copy link
Copy Markdown
Contributor

@echarrod echarrod commented Nov 7, 2025

Summary

  • Set toolchain to go1.25.3 in go.mod while keeping go directive at 1.25.0
  • Remove Go 1.24 from CI test matrix
  • Update all CI workflows to use go-version-file to target the version specified in go.mod

Changes

  • go.mod: Added toolchain go1.25.3
  • .github/workflows/test.yml: Removed Go 1.24 matrix, switched to go-version-file
  • .github/workflows/sonar.yml: Updated to use go-version-file instead of hardcoded version

All CI checks will now use the Go version defined in go.mod.

Summary by CodeRabbit

  • Chores
    • Go toolchain/version now derived from the project configuration to ensure consistent build environments.
    • CI build matrix simplified to reduce redundant multi-version testing.
    • CI now aligns Go version selection with the project’s module configuration for consistency.

- Set toolchain to go1.25.3 in go.mod while keeping go directive at 1.25.0
- Remove Go 1.24 from CI test matrix
- Update all CI workflows to use go-version-file to target go.mod version
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Nov 7, 2025

Walkthrough

The PR updates CI and toolchain configuration: both GitHub Actions workflows now use setup-go with go-version-file: 'go.mod' instead of hardcoded or matrix-provided versions; the test workflow’s matrix strategy was removed. go.mod gains a toolchain go1.25.3 directive and the go directive is set to 1.25. No changes to exported/public declarations or application code logic are present.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

  • Check GitHub Actions YAML for correct setup-go input syntax and that removal of the matrix doesn't unintentionally reduce coverage across Go versions.
  • Verify CI runners and any workflow steps that previously relied on the matrix still behave as expected.
  • Confirm toolchain go1.25.3 in go.mod is valid for local and CI toolchains.

Possibly related PRs

Poem

🐇 I found a mod and planted a rule,
One tiny line to keep versions cool.
Workflows now read where the version lives,
I bounce through CI with tidy little gives. 🥕

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main changes: updating Go toolchain to 1.25.3 and removing Go 1.24 compatibility from the test matrix.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-go-1.25.3

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ba9b58e and a09e41a.

📒 Files selected for processing (2)
  • .github/workflows/sonar.yml (1 hunks)
  • go.mod (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
  • .github/workflows/sonar.yml
  • go.mod

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@echarrod echarrod changed the title Update Go toolchain to 1.25.3 and remove 1.24 compatibility gomod: Update Go toolchain to 1.25.3 and remove 1.24 compatibility Nov 7, 2025
Changed go directive from 1.25.0 to 1.25 to allow setup-go to install
a version compatible with the toolchain go1.25.3 directive. This prevents
the 'compile: version does not match go tool version' error when running
tests with coverage instrumentation in CI.
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Nov 7, 2025

@echarrod echarrod merged commit 6b200e2 into main Nov 7, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant