Gradle - set a default value for baseline

[BraisGabin]

This fix #1260

Now that we don't create baselines when we don't need them and we allow empty baselines we can fix this old issue.

• It adds baseline.xml as default value in our gradle plugin
• And it make the baseline names of multiplatform a bit more consistents. This is a breacking change but as far as I know this part is experimental so it should not be a big problem.

[picklebento]

Note: I'll remove the "skip" commit before moving this out of Draft

The gradle integration test for multiplatform - iOS is also failing for me locally. We could skip this for local development if we don't have time to figure it out.

[picklebento]

Ideally, we should use something like

    var baseline: RegularFileProperty =
        objects.fileProperty().convention { File("baseline.xml") }

but this is a breaking change.

[BraisGabin]

I couldn't make it work with convention but now it works with gradle 5.4

[codecov]

Codecov Report

Merging #3619 (aca4bae) into main (709f879) will decrease coverage by 0.08%.
The diff coverage is 0.00%.

❗ Current head aca4bae differs from pull request most recent head cc747d8. Consider uploading reports for the commit cc747d8 to get more accurate results

@@             Coverage Diff              @@
##               main    #3619      +/-   ##
============================================
- Coverage     78.12%   78.03%   -0.09%     
  Complexity     2837     2837              
============================================
  Files           467      467              
  Lines          9143     9153      +10     
  Branches       1737     1742       +5     
============================================
  Hits           7143     7143              
- Misses         1058     1068      +10     
  Partials        942      942              

Impacted Files	Coverage Δ	Complexity Δ
...ab/arturbosch/detekt/extensions/DetektExtension.kt	0.00% <0.00%> (ø)	0.00 <0.00> (ø)
.../arturbosch/detekt/internal/DetektMultiplatform.kt	0.00% <0.00%> (ø)	0.00 <0.00> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 709f879...cc747d8. Read the comment docs.

[BraisGabin]

Could we move up our gradle compatibility? We are supporting a version from 16/Apr/2019

I think that we could move to 6.0. It was release at 08/Nov/2019.

[picklebento]

Agreed. We should be able to update README.md and the website.

[remcomokveld]

Given that detekt is not the only static analysis tool that has a baseline, would it make sense to rename the default to detekt-baseline.xml? Especically for someone new on a project it would make it easier to identify what this file is intended for.

[cortinico]

would it make sense to rename the default to detekt-baseline.xml?

Yup that's a fair point @remcomokveld 👍
We're addressing it here #3745