Makes dependency on Java check configurable

[gdecaso]

As discussed in #79, though not strictly related to that issue, we can relax the dependency from Java's check task to spotlessCheck.

For backwards compatibility, and to respect @nedtwigg 's opinion on the philosophy of this project, the default is to add the dependency.

[nedtwigg]

The "check" task isn't java-specific, it's meant to be general purpose.

Rather than living in "JavaExtension", the flag should live in "SpotlessExtension", and it should apply to all formatting tasks.

I think "runSpotlessCheckOnCheck" is unhelpful for users who haven't dug into the code. Maybe "enforceCheck"?

Also, do you mind squashing these into one commit for easier review? Thanks!

[gdecaso]

Thanks @nedtwigg. Applied your suggestions and squashed everything into 0711a77. Please let me know what you think

[gdecaso]

Thanks @jbduncan. Applied your suggestions. Please take a look at squashed commit 6dec185.

[gdecaso]

@jbduncan I don't think the rename is needed. I have a project where I'm printing the property just for testing purposes and I'm accessing as follows

spotless {
	enforceCheck = false
	println enforceCheck
}

[nedtwigg]

General types should be getX/setX, but booleans are a special case: isX/setX.

Thanks for a good PR :)

[gdecaso]

Thanks @nedtwigg. Please let me know if you are planning to do a release soon so I can uptake this in my projects. Or alternatively, do you have a Maven repository where this is in snapshot form?

[nedtwigg]

It's available in 3.2.0-SNAPSHOT here, and we'll release later today: https://oss.sonatype.org/content/repositories/snapshots/com/diffplug/spotless/spotless-plugin-gradle/

[gdecaso]

Thanks!