Skip to content

Comments

Follow the new way of injecting the version into the binary#73

Merged
DavidS-ovm merged 2 commits intomainfrom
ServiceVersion
Jan 7, 2025
Merged

Follow the new way of injecting the version into the binary#73
DavidS-ovm merged 2 commits intomainfrom
ServiceVersion

Conversation

@DavidS-ovm
Copy link
Contributor

@DavidS-ovm DavidS-ovm removed the request for review from dylanratcliffe January 7, 2025 14:55
@dylanratcliffe
Copy link
Member

Good catch. I don't see why the tests are failing though. That definitely looks like the right modification

@DavidS-ovm
Copy link
Contributor Author

Ah, this is still trying to build the 1.6.0 tag which does require the go generate hook. Will reinstate that until the next CLI release

@DavidS-ovm DavidS-ovm merged commit 9f8b763 into main Jan 7, 2025
5 checks passed
@DavidS-ovm DavidS-ovm deleted the ServiceVersion branch January 7, 2025 19:25
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