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

Display app versions #43

Closed
rodionovd opened this issue Sep 17, 2016 · 4 comments
Closed

Display app versions #43

rodionovd opened this issue Sep 17, 2016 · 4 comments

Comments

@rodionovd
Copy link
Contributor

list command

Current output:

409203825 Numbers
847496013 Deckset
1024069033 Veertu
452840086 Receigen
409201541 Pages

Proposed output:

409203825 Numbers (3.6)
847496013 Deckset (1.6.2)
1024069033 Veertu (1.1.4)
452840086 Receigen (3.4.0)
409201541 Pages (5.6)

outdated command

Current output:

409203825 Numbers (3.6.2)
409201541 Pages (5.6.2)

Proposed output:

409203825 Numbers (3.6 -> 3.6.2)
409201541 Pages (5.6 -> 5.6.2)

I'm working on a PR to implement this feature.
Question: Should I hide this feature under a new flag (e.g. --version) or just make it a default behaviour for both commands?

@argon
Copy link
Contributor

argon commented Sep 17, 2016

Just make it the default behaviour, I think

—A

On 17 Sep 2016, at 06:30, Dmitry Rodionov notifications@github.com wrote:

list command

Current output:

409203825 Numbers
847496013 Deckset
1024069033 Veertu
452840086 Receigen
409201541 Pages
Proposed output:

409203825 Numbers (3.6)
847496013 Deckset (1.6.2)
1024069033 Veertu (1.1.4)
452840086 Receigen (3.4.0)
409201541 Pages (5.6)
outdated command

Current output:

409203825 Numbers (3.6.2)
409201541 Pages (5.6.2)
Proposed output:

409203825 Numbers (3.6 -> 3.6.2)
409201541 Pages (5.6 -> 5.6.2)
I'm working on a PR to implement this feature.
Question: Should I hide this feature under a new flag (e.g. --version) or just make it a default behaviour for both commands?


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@rodionovd
Copy link
Contributor Author

OK, will do.

Oh, I'll probably postpone the PR until #44 is merged (I haven't even upgraded to Xcode 8 yet)

@argon
Copy link
Contributor

argon commented Sep 17, 2016

I'm happy to wait for your PR to merge #44. I'm still improving some things. it's a work in progress.

@rodionovd
Copy link
Contributor Author

Implemented in #45

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

No branches or pull requests

2 participants