Add an ignoreFailures property

Like pmd, checkstyles, findbugs plugins.

when ignoreFailures = true the build doesn’t fail.

spotless {
   ignoreFailures = true
  ...
}

[nedtwigg]

I am not against formatting always. I am against checking the formatting always.

If you format always, then the check will always pass 😃

if the plugin was such that Java’s check dependsOn spotlessApply

Then I could commit code with formatting errors, upload it to CI, and it would pass CI. And the commit history would be hard to read, because there would be meaningless formatting changes obscuring the changes that matter.

This commit has 16 changed lines. 2 of them matter, the other 14 are formatting noise which makes it harder to review the content of the change. If check depends on spotlessApply, CI cannot catch this.

[nedtwigg]

Formatting is not important. The point of Spotless is to take that discussion off the table - this is how this project is formatted, period, so let’s talk about the content instead of the formatting.

You’re looking at Spotless, so you’ve decided that you’d like for your code to be formatted a specific way.

You’re opening a new discussion - when should my code be formatted the way I want it to be formatted? to which there are three possible answers:

1. Never (if you’re using Spotless, then that’s not what you want)
2. Sometimes
3. Always

Always is simple. Formatting is not important, so I think the simple answer is the best answer, and the right default for Spotless.

Sometimes is complicated. If you have a compelling need to implement your particular Sometimes policy, Spotless supports that with spotlessCheck.dependsOn.remove('check') EDIT NedTwigg June 30 2020

spotless {
    enforceCheck = false
}