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

JSON Output Support #88

Merged
merged 2 commits into from
Mar 17, 2024
Merged

Conversation

etagwerker
Copy link
Contributor

Hi there,

This PR adds support for JSON output. It fixes #86.

In order to make it easier to test the new format, I created a Reporter class that knows how to process the output format options.

Please check it out and let me know what you think.

Thanks,
Ernesto

@etagwerker
Copy link
Contributor Author

Once #87 is merged to the main branch, this PR's test suite should pass as well. I tested that locally and I'm quite confident it just needs the changes from #87 to pass.

@danmayer danmayer merged commit b76db8e into danmayer:master Mar 17, 2024
0 of 14 checks passed
@danmayer
Copy link
Owner

thanks updated these and renamed the default branch from master to main

@etagwerker etagwerker deleted the fixes-86-json-support branch March 17, 2024 23:48
@hemalvarambhia
Copy link

@danmayer Hello there. I am experimenting with this gem at the moment, and a JSON output would be useful. When could this work be released? I also notice that the gem is very fast compared to how rubycritic computes churn, which made me wonder whether rubycritic could integrate your gem ❤️ .

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 this pull request may close these issues.

Feature request: Add support for json output
3 participants