Sanitize gradle build scripts and suppress unstable api usages

[arturbosch]

No description provided.

[schalkms]

I’m not sure, whether we should suppress these issues.
Why did you remove the BuildFailure exception?

[codecov-io]

Codecov Report

Merging #2271 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2271      +/-   ##
============================================
+ Coverage     81.61%   81.65%   +0.03%     
+ Complexity     2120     2110      -10     
============================================
  Files           349      348       -1     
  Lines          6054     6033      -21     
  Branches       1106     1105       -1     
============================================
- Hits           4941     4926      -15     
+ Misses          524      518       -6     
  Partials        589      589

Impacted Files	Coverage Δ	Complexity Δ
...ain/kotlin/io/gitlab/arturbosch/detekt/cli/Main.kt	36% <0%> (-14%)	6% <0%> (-1%)
.../io/gitlab/arturbosch/detekt/cli/IssueExtension.kt	87.09% <0%> (-6.46%)	0% <0%> (ø)
...in/io/gitlab/arturbosch/detekt/cli/OutputFacade.kt	94.11% <0%> (-0.62%)	6% <0%> (-3%)
.../io/gitlab/arturbosch/detekt/cli/runners/Runner.kt	87.71% <0%> (ø)	15% <0%> (ø)	⬇️
...tlin/io/gitlab/arturbosch/detekt/generator/Main.kt	0% <0%> (ø)	0% <0%> (ø)	⬇️
...ekt/generator/printer/rulesetpage/ConfigPrinter.kt	76.74% <0%> (ø)	8% <0%> (ø)	⬇️
...rturbosch/detekt/cli/console/BuildFailureReport.kt
...in/io/gitlab/arturbosch/detekt/cli/BuildFailure.kt
...n/kotlin/io/gitlab/arturbosch/detekt/cli/Errors.kt	66.66% <0%> (ø)	0% <0%> (?)
...lab/arturbosch/detekt/core/FileProcessorLocator.kt	84.61% <0%> (+7.69%)	4% <0%> (+2%)	⬆️
... and 1 more

---

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 81494bf...6233e39. Read the comment docs.

[arturbosch]

I’m not sure, whether we should suppress these issues.

Yeah maybe not, this way we will see when the api gets stable. I revert them.

Why did you remove the BuildFailure exception?

It was only used in the ProcessExecutor which is again only used by the Idea wrapper tasks. Gradle will treat it as a build failure for us. I think it is legacy code from before the gradle rewrite to dynamic resolution.