Skip to content

Allow the car binary to provide a version. - #645

Merged
willscott merged 1 commit into
masterfrom
feat/version
Feb 18, 2026
Merged

Allow the car binary to provide a version.#645
willscott merged 1 commit into
masterfrom
feat/version

Conversation

@willscott

Copy link
Copy Markdown
Member

This is done by running go generate before go build.

This hopefully addresses the request in #644 in a minimally invasive way.

This is done by running `go generate` before `go build`.
Comment thread cmd/car/version.txt
@@ -0,0 +1 @@
v2.x.x-unknown

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Shouldn't this be set to the latest tagged version, which is currently v2.16.0?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It's a placeholder that allows go run ... to work.
in general, go generate will update this so that the result of builds will have a correct version. Since there are both v1 and v2 tags that will apply, i think it's going to be cleaner to not keep each tag leading to file changes that need to be checked in, and only have a template placeholders as the committed thing in the repo

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sounds good, thank you.

@willscott
willscott merged commit 5383c06 into master Feb 18, 2026
16 of 17 checks passed
@willscott
willscott deleted the feat/version branch February 18, 2026 06:42
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