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

Include git revision of StackStorm release in the st2 --version output when running a dev release #4117

Merged
merged 4 commits into from
May 8, 2018

Conversation

Kami
Copy link
Member

@Kami Kami commented May 8, 2018

This pull request updates st2 --version CLI command to also include git revision of the release when running a dev release and when file with package metadata is available.

Before:

$ st2 --version
st2 2.8dev, on Python 2.7

After:

$ st2 --version
st2 2.8dev (5f89c1f), on Python 2.7.6

This saves some typing since you don't need to manually check the package metadata file (/opt/stackstorm/st2/package.meta).

If others think it makes sense, we could also include git revision when running a stable (non dev) release, but I personally think that's not needed and it would just make it confusing (we have a corresponding git tag for each final release which is not the case for dev releases).

TODO

  • Tests

"st2 --version" CLI command and when running a dev release.
@Kami Kami added this to the 2.8.0 milestone May 8, 2018
Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

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

👍 Good one!

I agree that it makes sense for dev version only.

@Kami Kami merged commit 92e5823 into master May 8, 2018
@Kami Kami deleted the st2_version_include_git_commit_sha_on_dev branch May 8, 2018 17:05
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