CollectViolations is skipped when xml is not available
tasomaniac opened this issue · 1 comments
tasomaniac commented
To be able to collect the number of errors and warnings and then fail the build accordingly, we really depend on availability of xml reports.
At least for detekt, when xml report is disabled, we just ignore/skip. This causes the build to be successful even when there are errors.
We should warn user in configuration time and ask them to enable reports.
tasomaniac commented
I see that the tools might skip creating the report file when there are no errors/warnings to report. That's why we have onlyIf {} in there. So this is working as expected.