Skip to content
This repository has been archived by the owner on Jun 28, 2023. It is now read-only.

Use buildinfo.Version instead of buildVersion when building with tanzu framework v0.10.0 #2523

Merged
merged 2 commits into from
Nov 15, 2021

Conversation

jpmcb
Copy link
Contributor

@jpmcb jpmcb commented Nov 11, 2021

What this PR does / why we need it

This uses the newer buildinfo.Version api in tanzu framework instead of the cli.buildVersion api (which has been deprecated vmware-tanzu/tanzu-framework@3162254#diff-287b2c13779f4ba2d8e41cc7c201cb67605ab8ef8cce54a977df7a5c83b00dddR20-R21)

Details for the Release Notes (PLEASE PROVIDE)

N/a - This is more to demonstrate using the new API so we can bump our tanzu framework version

Which issue(s) this PR fixes

Related to #2520

Describe testing done for PR

Able to make install-all-tanzu-cli-plugins

Special notes for your reviewer

See note below on the standalone-cluster version

@jpmcb jpmcb requested a review from a team as a code owner November 11, 2021 18:44
@github-actions github-actions bot added area/cli owner/release-eng Work executed by VMware release engineering team labels Nov 11, 2021
@jpmcb jpmcb marked this pull request as draft November 11, 2021 18:47
@jpmcb jpmcb force-pushed the david-update-frame branch from 919914f to 623309d Compare November 15, 2021 17:45
@jpmcb jpmcb marked this pull request as ready for review November 15, 2021 17:45
@jpmcb jpmcb changed the title WIP: Use buildinfo.Version instead of buildVersion when building with tanzu framework v0.10.0 Use buildinfo.Version instead of buildVersion when building with tanzu framework v0.10.0 Nov 15, 2021
davidvonthenen
davidvonthenen previously approved these changes Nov 15, 2021
Copy link
Contributor

@davidvonthenen davidvonthenen left a comment

Choose a reason for hiding this comment

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

woot woot!

Replaces using buildVersion ldflag

Signed-off-by: John McBride <[email protected]>
@jpmcb jpmcb force-pushed the david-update-frame branch from 7dcff4e to 826324a Compare November 15, 2021 19:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-not-required owner/release-eng Work executed by VMware release engineering team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants