send errors and warnings to stderr
This should help address https://github.com/linode/linode-cli/issues/643, although possibly only in part.
Overall looks good to me!
Hey @jschauma, thanks for the contribution! Could you run make format on this PR to fix the lint CI failure?
Sure - ran 'make format'. (I find ending the printf list of arguments with a comma a bit odd, but if that's what the formatter wants...?)
Sure - ran 'make format'. (I find ending the printf list of arguments with a comma a bit odd, but if that's what the formatter wants...?)
Thank you! The formatter can make some odd decisions at times but we try to keep things as consistent as possible.
FYI: I marked this PR as a breaking change since it technically could break scripts that check for error outputs in stdout. We'll mention this in the notes for the next release.
@zliang-akamai I've been bitten by the behavior fixed by this PR - all of my VM provisioning fails b/c I get an extra
Using default values: {}; use the --no-defaults flag to disable defaults
in the json output. Strangely enough version 5.52.0 doesn't fail for me.
I will add a workaround on my side, however do yu mind pushing a new release to PyPI which includes this PR?