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

Use `print_value()` instead of `log()` for output of `config get` #78

Merged
merged 1 commit into from Dec 12, 2018

Conversation

2 participants
@emirpprime
Copy link

emirpprime commented Dec 10, 2018

Move get command from ::log to ::print_value for the output so that --quiet doesn't suppress it.

Fixes #77

Phil Banks
::log -> ::print_value for output of get cmd
- ::log causes output to be lost when using --quiet

@schlessera schlessera added this to the 2.0.2 milestone Dec 12, 2018

@schlessera schlessera merged commit 9a28481 into wp-cli:master Dec 12, 2018

2 checks passed

DEP All dependencies are resolved
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@schlessera

This comment has been minimized.

Copy link
Member

schlessera commented Dec 12, 2018

Thanks for the PR, @emirpprime !

@schlessera schlessera changed the title ::log -> ::print_value for output of get cmd Use `print_value()` instead of `log()` for output of `config get` Dec 12, 2018

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