Gradle: how to run lint automatically with `check`? about ktlint-gradle HOT 7 CLOSED

We should support this. For now the solution is to do the following:

afterEvaluate {
    test.dependsOn ktlintCheck
}

from ktlint-gradle.

@mverleg feel free to reopen if it will not work for you.

from ktlint-gradle.

Thanks for the answer!

I'm afraid it still doesn't work. I can use the task directly, but putting the suggested code in the top-level build.gradle (near the bottom) still gives the same error.

(Using a string ':topproject:ktlintCheck' moves the error to runtime but doesn't work)

from ktlint-gradle.

@mverleg note, that if you have multimodule project, root project will not have test or check tasks. You can archive it with following:

subprojects {
    afterEvaluate {
        if (it.plugins.findPlugin("org.jlleitschuh.gradle.ktlint") != null) {
            check.dependsOn("ktlintCheck")
        }
    }
}

from ktlint-gradle.

Though I agree in general that check task should depend on ktlintCheck task.

from ktlint-gradle.

Actually plugin already set check task to depend on all ktlint check tasks.

For example:

$ ./gradlew -m :samples:kotlin-ks:check
:samples:kotlin-ks:ktlintMainCheck SKIPPED
:samples:kotlin-ks:ktlintTestCheck SKIPPED
:samples:kotlin-ks:compileKotlin SKIPPED
:samples:kotlin-ks:compileJava SKIPPED
:samples:kotlin-ks:processResources SKIPPED
:samples:kotlin-ks:classes SKIPPED
:samples:kotlin-ks:compileTestKotlin SKIPPED
:samples:kotlin-ks:compileTestJava SKIPPED
:samples:kotlin-ks:processTestResources SKIPPED
:samples:kotlin-ks:testClasses SKIPPED
:samples:kotlin-ks:test SKIPPED
:samples:kotlin-ks:check SKIPPED

Deprecated Gradle features were used in this build, making it incompatible with Gradle 5.0.
See https://docs.gradle.org/4.5/userguide/command_line_interface.html#sec:command_line_warnings

BUILD SUCCESSFUL in 0s

@mverleg can you check it for your project setup?

from ktlint-gradle.

Yeah I made a sample and it does have the dependency by default for JVM project, so I think this will mostly be fixed by #58 .

Though it's worth noting that

test.dependsOn ktlintCheck

still gives the error from the first post, even for JVM. It's not really a problem since it already happens by default, but it's a little weird.

This does work for JVM, but not for other platforms:

afterEvaluate {
    test.dependsOn ktlintCheck
}

from ktlint-gradle.