Formatting to JSON about xcbeautify HOT 5 CLOSED

Hey 👋

It’s current not possible to output to JSON format, and the current implementation might require a bit of refactoring in order to make the output extensible.

from xcbeautify.

@cpisciotta we're using Bitrise in combination with Danger, so we're still in need of a JSON output. However, we moved away from Xcode Summary, so there's no need for this issue from my perspective anymore. Feel free to close.

from xcbeautify.

Hey @AvdLee! I'm working on some changes (see here: #107) to introduce GitHub Actions support. My draft PR proposes a refactor where there'd be different OutputRendering types. So, the current implementation would change to a TerminalRenderer and then I'm adding another one called GitHubActionsRenderer. This new renderer would support inline feedback around warnings, errors, and messages, but it'd obviously be limited to PRs using GitHub Actions.

For your use case, I think it'd make more sense to build something similar to the JUnitReporter. However, this is making me question if it'd make sense to consolidate these different reporters and renderers, since it's likely that a consumer only cares about only one of the different options.

Are you using GitHub Actions? If so, I think you might be able to skip the need for JSON reporting altogether and use the renderer that I'm proposing. I have a similar setup and use case.

@AvdLee @pepicrft @thii Thoughts?

from xcbeautify.

@cpisciotta the approach you are suggesting sounds reasonable from y end.

from xcbeautify.

@AvdLee Got it- thanks for the update!

from xcbeautify.