Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Uses Golang runtime/debug package to get version info #158

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

voltagex
Copy link

I changed the version code to do this: https://github.com/voltagex-forks/dive/blob/eff28ef17026691aa6d4b49f2b893ec6115cb5f8/cmd/version.go

dive.exe -v
dive (devel) from github.com/jauderho/dive

without this patch:

.\dive.exe -v
dive No version provided

I suspect that once you tag and build it, it'll show the tag version/name.

This uses runtime/debug.

As requested, I split this out of #154

Copy link

semanticdiff-com bot commented Sep 30, 2024

Review changes with SemanticDiff.

Analyzed 1 of 1 files.

Overall, the semantic diff is 7% smaller than the GitHub diff.

Filename Status
✔️ cmd/version.go 6.27% smaller

Signed-off-by: Adam Baxter <git@voltagex.org>
Copy link

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant