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.

Sign up for GitHub

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

Jump to bottom

Implement '--version' #260

Closed
LinuxOnTheDesktop opened this issue Apr 18, 2019 · 4 comments · Fixed by #295
Closed

Implement '--version' #260

LinuxOnTheDesktop opened this issue Apr 18, 2019 · 4 comments · Fixed by #295
Labels
good first issue Simple change to start learning code base priority:low Nice to have feature, minor improvement to functionality or usability

Comments

@LinuxOnTheDesktop
Copy link

Describe the solution you'd like
I would like vorta to follow standard and useful practice in that I would like vorta --version to yield version information.
@m3nu
Copy link
Contributor

m3nu commented Apr 18, 2019

Can be added. But then most GUI apps don't have many CLI arguments.

@m3nu m3nu added good first issue Simple change to start learning code base priority:low Nice to have feature, minor improvement to functionality or usability labels Apr 18, 2019
@LinuxOnTheDesktop
Copy link
Author

But then most GUI apps don't have many CLI arguments.

True, but if one installs Vorta via downloading and moving the binary, then, unless one does other things (namely, adds Vorta to one's launchers), then one will be apt to launch Vorta from the commandline (though I suppose one should not do so).

Can be added.

Thanks.

@Hofer-Julian
Copy link
Collaborator

Actually, I also miss a version argument. Also CLI arguments are common for Linux apps (at least a lot of the apps I use have them).

@m3nu if it's okay for you, I can add it.

@m3nu
Copy link
Contributor

m3nu commented Apr 18, 2019

@m3nu if it's okay for you, I can add it.

Any time :-)

Hofer-Julian pushed a commit to Hofer-Julian/vorta that referenced this issue May 30, 2019
Implement --version
7c5499b 
Fixes borgbase#260
@Hofer-Julian Hofer-Julian mentioned this issue May 30, 2019
Merged
Hofer-Julian pushed a commit that referenced this issue May 30, 2019
Implement --version. Fixes #260 (#295)
23a92e3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Simple change to start learning code base priority:low Nice to have feature, minor improvement to functionality or usability
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Implement --version Hofer-Julian/vorta
3 participants
@m3nu @Hofer-Julian @LinuxOnTheDesktop