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

ui/ux(cli): create and update message #55

Closed
0x-r4bbit opened this issue Jul 13, 2015 · 4 comments · Fixed by #64
Closed

ui/ux(cli): create and update message #55

0x-r4bbit opened this issue Jul 13, 2015 · 4 comments · Fixed by #64

Comments

@0x-r4bbit
Copy link
Contributor

This is really just for better UX:

Whenever I create or update a changelog with clog the output I get is always:

$ changelog updated (took xx ms)

I think it'd be nicer if the tools states that it created a changelog, if that's the case and only updates if that's the case.

@kbknapp
Copy link
Member

kbknapp commented Jul 13, 2015

This is an easy fix, but with #58 I don't know if they're compatible. For example, if I do

$ clog > some_file.md

I may not want that file to contain "Updated/Created 15ms" ( now if we don't implement #58 this is a moot point and very simple to fix :P )

@0x-r4bbit
Copy link
Contributor Author

@kbknapp you're totally right. Does it make sense to generate that output only if a writer is applied? Not sure what's the standard way in UNIX commands to handle this..

@kbknapp
Copy link
Member

kbknapp commented Jul 13, 2015

@PascalPrecht Yes, that's good idea, only write this message when not writing to stdout.

@0x-r4bbit
Copy link
Contributor Author

@kbknapp I'd like to keep as UNIX-y as possible. If that's how all UNIX commands handle these cases, I'm voting for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants