Skip to content

goreleaser: embed VERSION#153

Merged
thepwagner merged 1 commit into
mainfrom
rm-version-go
May 28, 2024
Merged

goreleaser: embed VERSION#153
thepwagner merged 1 commit into
mainfrom
rm-version-go

Conversation

@thepwagner

Copy link
Copy Markdown
Contributor

Replace the logic in Makefile that replaces cmd/ejson/version.go with the current tag.

It's more common to inject these variables during linking, instead of generating source code.

Verify like:

$ cat cmd/ejson/version.go
package main

var VERSION string = "1.5.0"

$ goreleaser build --snapshot --rm-dist

$ dist/ejson_darwin_arm64/ejson --version
ejson version 1.5.1-SNAPSHOT-1ecf7d7

@thepwagner
thepwagner requested a review from burke as a code owner May 24, 2024 16:38
@thepwagner
thepwagner merged commit d8c74dd into main May 28, 2024
@thepwagner
thepwagner deleted the rm-version-go branch May 28, 2024 17:07
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