Skip to content

Update Go version to 1.25.*#8

Merged
nDmitry merged 2 commits intomainfrom
go-update
Dec 6, 2025
Merged

Update Go version to 1.25.*#8
nDmitry merged 2 commits intomainfrom
go-update

Conversation

@nDmitry
Copy link
Copy Markdown
Owner

@nDmitry nDmitry commented Dec 6, 2025

Summary by CodeRabbit

  • Chores
    • Updated Go toolchain from 1.24 to 1.25 across build and module configurations to align with the newer language version.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Dec 6, 2025

Walkthrough

Go toolchain version is upgraded from 1.24 (or 1.24.0) to 1.25 in the Dockerfile, go.mod, and go.tool.mod. Additionally, go.tool.mod updates the module path from gitlab.pharmhub.ru/.../w-privileges/tools to github.com/nDmitry/tgfeed/tools. No other functional changes.

Changes

Cohort / File(s) Summary
Dockerfile (build image arg)
Dockerfile
Bumped ARG GO_VERSION from 1.24 to 1.25; build steps unchanged.
Module Go version
go.mod
Updated module Go version from 1.24.0 to 1.25. No other edits.
Tool module manifest
go.tool.mod
Changed module path from gitlab.pharmhub.ru/pharmhub/pharmcode/apps/w-privileges/tools to github.com/nDmitry/tgfeed/tools and bumped go from 1.24 to 1.25; tool directives unchanged.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Review focus:
    • Verify go.tool.mod module path change is intentional and compatible with repo structure and CI.
    • Confirm no implicit dependency/tooling expectations break due to Go version bump (build scripts, CI images).
    • Quick sanity check of Dockerfile build image tag consistency.

Poem

🐰 I hopped through lines and gave a tiny shove,
From twenty-four I leapt to twenty-five with love,
New module name, a fresh small springtime cheer,
Built and bundled — nimble paws, no fear! 🥕✨

Pre-merge checks

✅ 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 'Update Go version to 1.25.*' directly and clearly summarizes the main change across all modified files (Dockerfile, go.mod, and go.tool.mod), which consistently update the Go version from 1.24 to 1.25.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 68daef5 and 99ae83d.

📒 Files selected for processing (3)
  • Dockerfile (1 hunks)
  • go.mod (1 hunks)
  • go.tool.mod (1 hunks)
🔇 Additional comments (2)
go.tool.mod (1)

3-3: Go version update is consistent with go.mod.

The Go version update to 1.25 matches the change in go.mod. However, this assumes the module path issue (Line 1) is resolved first.

Dockerfile (1)

3-3: Dockerfile update is consistent with module files.

The ARG GO_VERSION update to 1.25 aligns with the changes in go.mod and go.tool.mod, ensuring the build uses the correct Go toolchain version. This assumes that the golang:1.25 Docker image exists and Go 1.25 is released (see verification request in go.mod review).

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 99ae83d and bd44b73.

📒 Files selected for processing (1)
  • go.tool.mod (1 hunks)
🔇 Additional comments (1)
go.tool.mod (1)

1-1: Module path issue resolved.

The module path has been correctly updated to match the repository. This resolves the critical issue flagged in the previous review.

@nDmitry nDmitry merged commit e2d76b0 into main Dec 6, 2025
3 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