Skip to content

Update GoReleaser documentation and installation command - #125

Merged
besendorf merged 2 commits into
mvt-project:mainfrom
sealad886:fix-goreleaser-version
Sep 1, 2026
Merged

besendorf merged 2 commits into
mvt-project:mainfrom
sealad886:fix-goreleaser-version

Conversation

@sealad886

Copy link
Copy Markdown
Contributor

This pull request updates the GoReleaser installation instructions to use the correct Go module path, ensuring compatibility with Go modules v2 and above.

Documentation and script updates:

  • Updated the GoReleaser installation command in the README.md to use github.com/goreleaser/goreleaser/v2@v2.17.1 instead of the deprecated path.
  • Updated the installation suggestion in build_locally.sh to match the correct GoReleaser module path.

Version 2 of goreleaser is indexed one folder deeper in the source repo.
Maintain consistency in help text for failed goreleaser execution
Copilot AI lite review requested due to automatic review settings September 1, 2026 12:28

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The changes are small, consistent across the docs and script, and correctly update the GoReleaser module path without affecting build logic.

Pull request overview

Updates local build documentation and helper script messaging to use the correct GoReleaser v2 Go module path (.../v2@...), aligning installation instructions with Go modules major-version import semantics.

Changes:

  • Updated README.md GoReleaser install command to github.com/goreleaser/goreleaser/v2@v2.17.1.
  • Updated build_locally.sh installation hint to match the corrected module path.
File summaries
File Description
README.md Fixes the documented go install command to use the v2 module path for GoReleaser.
build_locally.sh Aligns the “not installed” guidance message with the corrected GoReleaser v2 install path.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@besendorf besendorf left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Reviewed both changed files. The corrected GoReleaser v2 module path resolves at v2.17.1, while the previous path fails Go module major-version validation. No findings.

@besendorf
besendorf merged commit c7159de into mvt-project:main Sep 1, 2026
1 check 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.

3 participants