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

Remove warnings when using JSON formatter #576

Closed
kalkov opened this issue Oct 19, 2013 · 4 comments · Fixed by #597
Closed

Remove warnings when using JSON formatter #576

kalkov opened this issue Oct 19, 2013 · 4 comments · Fixed by #597
Assignees

Comments

@kalkov
Copy link

kalkov commented Oct 19, 2013

Hi, while parsing the json from json_formatter I get warnings like this :

  Warning: unrecognized cop Output found in

Which breaks the parsing. Don't you think it's good idea to put them in the json?

@yujinakayama
Copy link
Collaborator

I guess you are using --format json without --out somefile (so the JSON data will be outputted to the STDOUT) and the problem is that config validation warning is also outputted to the STDOUT.

I think the warning should be outputted to STDERR so that your JSON parser need to just take care of STDOUT.

@kalkov
Copy link
Author

kalkov commented Oct 20, 2013

You are right, I'm not outputting to a file. It will be nice to have warnings in the JSON itself.

@ghost ghost assigned yujinakayama Oct 25, 2013
@yujinakayama
Copy link
Collaborator

I think there are some technical issues with storing warnings in JSON and actually I wonder if warnings really should be in JSON. So for now, I'll change the warning to be outputted to STDERR.

@kalkov
Copy link
Author

kalkov commented Nov 4, 2013

Thanks

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

Successfully merging a pull request may close this issue.

2 participants