Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Feb 24, 2020. It is now read-only.

Conversation

@achanda
Copy link
Contributor

@achanda achanda commented Jan 21, 2016

No description provided.

@alban
Copy link
Member

alban commented Jan 21, 2016

/cc @blixtra @robszumski

@jonboulle
Copy link
Contributor

Can you help me understand your motivation for adding this? If we do, it should be stderr (recently wrote up my rationale here)

@jonboulle
Copy link
Contributor

Assuming this is related to #2011 - if you have an example of trying to integrate this with Ansible or some other system, please provide details so we can better understand your use case.

@achanda
Copy link
Contributor Author

achanda commented Jan 22, 2016

Honestly, this was pretty ad-hoc, with no defined use case. I thought conveying more information to an user will lead to better UX.

@jonboulle
Copy link
Contributor

I'm not necessarily vetoing that, just saying it should be on stderr if so.

@achanda
Copy link
Contributor Author

achanda commented Jan 23, 2016

Changed to stderr

@jonboulle
Copy link
Contributor

I feel weakly about this so going to defer to one of the other maintainers.

@achanda achanda closed this Feb 4, 2016
@achanda
Copy link
Contributor Author

achanda commented Feb 4, 2016

Superseded by #2101

@achanda achanda deleted the report_progress branch February 4, 2016 22:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants