Same output file for every task

[jlleitschuh]

Same output file for every task

[originx]

should we do it in this PR or a new one?
@leinardi what does the change affect?

#32
@Tapchicoma

[leinardi]

@originx I think a new one is better. The changes are not in the method touched by #32.
But before creating one I'd like to hear from @Tapchicoma or @JLLeitschuh if they are interested in this.

[JLLeitschuh]

I don't personally use the output report.

I'd say that we should make the output configurable on the task but keep the default a sane configuration.

[Tapchicoma]

It is a good idea, though checkstyle, for example, doesn't do this. It just aggregates everything in one file. Also this is more Android specific, though Gradle start to introduce SoftwareModel that will allow to have flavors on the plain Kotlin projects.

Also this feature should be configurable and disabled by default.

[leinardi]

With the current implementation if you run different tasks the output will be always the same file, and this file will be overwritten every time and not merged automatically. So, currently, you loose the report of all the tasks but the last executed.

I don't see what is the benefit of having it disabled by default. It will be useful if you execute different tasks and nothing will change if you only have one beside the name of the file itself.

The file is generate only for the tasks that you are actually executing and not for every variant you have.

In my example I have dozens of variants but I only run two tasks: ktlintNonpayItalyCompatDebugCheck and ktlintNonpayMexicoCompatDebugCheck.

[leinardi]

OK, I fear I've been misunderstood: I don't want to enable anything new related to the report output, I just want to generate the file name for the output in a different way. To make it more clear I have opened a pull request with my changes. This should have zero impact on non Android builds beside changing the name from reports/ktlint/ktlint.xml to reports/ktlint-reports-main.xml.

@Tapchicoma, @JLLeitschuh please have a look and if you have questions just let me know.

[Tapchicoma]

Closed via #22