Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

print version info from git describe #30

Merged
merged 6 commits into from
May 19, 2023

Conversation

danlangford
Copy link
Collaborator

this small change adds a new line to the BMAI banner that includes some version information.

BMAI: the Button Men AI
Copyright (c) 2001-2023, Denis Papp.
For information, contact Denis Papp, denis@accessdenied.net
Version: v3.0-30-gb41f92b

v3.0 = the most recent git tag on the current branch
30 = how many commits have been made AFTER that most recent tag
g = git scms
b41f92b = commit hash that build was done from

this will allow somebody to know if they are running the latest version and help them id exactly what state the code was in when compiled

@danlangford danlangford requested a review from pappde May 19, 2023 19:43
@danlangford
Copy link
Collaborator Author

the git hash is a little wonky on PRs. PRs make an invisible branch behind the scenes with its own hash thats not visible in this ui. I think. in hoping if this gets merged into main the builds off main will have the hash that we know and love from the GitHub ui. this is a little trial and error that I cannot test without getting on a main branch. I could have tested it out on my fork now that I think about it… if you want me to I can.

if we merge into main and its garbage ill work out another solution

@pappde pappde merged commit 51d8cc8 into pappde:main May 19, 2023
@pappde
Copy link
Owner

pappde commented May 19, 2023

Neat idea.

@danlangford danlangford deleted the version-git-hash branch May 19, 2023 21:54
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