Allow to print Junit report in human-readable format

shipmonk-rnd / composer-dependency-analyser

🚀 Fast detection of composer dependency issues (unused dependencies, shadow dependencies, misplaced dependencies)

MIT License

332 stars 8 forks source link

Allow to print Junit report in human-readable format #154

Closed llaville closed 2 days ago

llaville commented 1 week ago

Fixes #153

llaville commented 1 week ago

Introducing the new class ShipMonk\ComposerDependencyAnalyser\Result\AbstractXmlFormatter is in goal context to propose a checkstyle output format support

JanTvrdik commented 1 week ago

I don't think we need such option. We should just always pretty-print the output.

llaville commented 1 week ago

I don't think we need such option. We should just always pretty-print the output.

I don't agree with you : my opinion is that

even if pretty print format has a ratio size to 20% greater than compact version, it's only used for debugging purpose by a human.
when you use with a third-party tool/platform, machine-readable (compact) version is enough.

llaville commented 2 days ago

Closing this PR because we do not have the same vision of reporting format and usage. @janedbal Thanks to spent time for code review

BTW, there are other enhancement that should be fixed first, to allow re-usability of your reports, otherwise no integration with third-party tools may be thinking .

janedbal commented 2 days ago

there are other enhancement that should be fixed first, to allow re-usability of your reports

Anything else beside https://github.com/shipmonk-rnd/composer-dependency-analyser/issues/156 and things you already reported?

llaville commented 1 day ago

Yes and https://github.com/shipmonk-rnd/composer-dependency-analyser/issues/157#issuecomment-2220470594