Allow configuration of when to run spotless in gradle build about spotless HOT 8 CLOSED

as a matter of fact you are totally right both in "when the task is called" and in the statement that spotless is much faster ;-)

from spotless.

If you'd like to remove spotlessCheck from the check task, this will do it:

afterEvaluate {
    tasks.getByName('check').dependsOn.remove(tasks.getByName('spotlessCheck'))
}

We're going to keep the default behavior of adding spotlessCheck to check automatically. A positive value in the gradle ecosystem is "convention over configuration" - when you add unit tests, style checkers, static analysis tools, they all hook into check by default, so that you don't have to do the plumbing yourself. Perfectly valid that you'd like to use it a different way though :)

from spotless.

Thanks for the workaround, good enough for our case. 👍
Maybe adding this to the Readme.md would help other people too.

from spotless.

Maybe adding this to the Readme.md would help other people too.

It's a good question, and GitHub's issue search is very good. Hopefully others who have this same issue will find this question.

I try to make the readme describe the project as concisely as possible. I believe this use case is a less <10% case, and thus doesn't earn a spot. As evidence, if your use case was more common, then every stylechecker / static analyzer / unit test runner in the gradle ecosystem should have this in their readme.

from spotless.

Also, out of curiosity, why do you not want spotlessCheck to run on every call to check?

from spotless.

We like to separate concerns in the build on CI
e.g. compiling/packaging <-> running unit tests <-> running integration tests <-> auditing code.

Also, we like to keep the build as fast as possible during development to achieve small turnaround times.

from spotless.

Here's gradle's task dependency graph:

If you'd like to just compile and package, run assemble. If you just want to run tests, run test (or integTest if you've made a separate unit test task).

The objective of the check task is (according to the gradle docs):

All verification tasks in the project, including test. Some plugins add additional verification tasks to the project.

You can use Gradle however you'd like, but it sounds like you're swimming upstream :) If you want people to always be able to run gradle build and have it be fast, I'd recommend that you instead print out the task graph above and leave it on their desk, and then they'll know which tasks to run for themselves, depending on how fast they'd like their result.

Also, spotless is much faster than a reasonable unit test suite :)

from spotless.

For anyone else stumbling upon this from Google: this has been added in the meantime.
To remove spotlessCheck as a dependency to check when using the gradle plugin, add

spotless {
    isEnforceCheck = false
}

See the plugin-gradle/README.md

Background

from spotless.