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

supermarket-ctl should have a version option #1805

Closed
xorima opened this issue Apr 23, 2019 · 1 comment · Fixed by #1811
Closed

supermarket-ctl should have a version option #1805

xorima opened this issue Apr 23, 2019 · 1 comment · Fixed by #1811
Labels
Component: Omnibus Involving the omnibus package or its build Priority: Low Status: Good First Issue An issue ready for a new contributor. Triage: Confirmed Indicates and issue has been confirmed as described. Type: Bug Does not work as expected.

Comments

@xorima
Copy link

xorima commented Apr 23, 2019

All other chef software supports the version flag, I would expect this to be consistent across all chef binaries

Describe the problem

As a user I want to see the version of supermarket I am running

Replication Case

run supermarket-ctl version

Possible Solution

add in version flag

@robbkidd robbkidd added Aspect: Correctness Component: Omnibus Involving the omnibus package or its build Priority: Low Triage: Confirmed Indicates and issue has been confirmed as described. Type: Bug Does not work as expected. Status: Good First Issue An issue ready for a new contributor. labels Apr 23, 2019
@robbkidd
Copy link
Contributor

A supermarket-ctl version command would like a lot like chef-server-ctl version but in Supermarket's omnibus/cookbooks/omnibus-supermarket/files/default/ctl-commands.

I'm going to mark this as a decent first-time contribution. I'll help someone contribute this, if there is interest. I'm also willing to be told I'm deluded and that this is not a good first issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Omnibus Involving the omnibus package or its build Priority: Low Status: Good First Issue An issue ready for a new contributor. Triage: Confirmed Indicates and issue has been confirmed as described. Type: Bug Does not work as expected.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants