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
Add a json reporter #2582
Add a json reporter #2582
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for this PR! Could you maybe also add some unit tests?
Ok, finally found some time to fix the remarks: Added some unit tests, removed the indention option and adjusted some things you commented on in the analog pr #2601 which were also present in my pr. I've also added the option for a custom filename besides the custom path. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great overall! I've got one more remark, please take a look if you have some time. If not, I can merge it in and change it before doing a release. Your choice 👍
Thanks! 👍 |
Many thanks for the feature @Tummerhore! |
Fixes #1814. The implementation is relatively simple and copies most of the code from the html reporter. Probably this can be refactored at some point so that there is less duplicated logic. But it works for now.