Add DSL for honoring asserts about gradle-nullaway-plugin HOT 8 CLOSED

It's in v0.2 already actually (https://github.com/tbroyer/gradle-nullaway-plugin/blob/v0.2/src/main/kotlin/net/ltgt/gradle/nullaway/NullAwayOptions.kt#L33), though it looks like I forgot to document it in the README.

from gradle-nullaway-plugin.

Fixed in 1babda7

Note that, looking back at the code, I'll soon rename the is-prefixed properties:

• isAssertsEnabled and isExhaustiveOverride will lose their is prefix in the Groovy DSL (but still have it with the Kotlin DSL)
• isSuppressSuggestions will be renamed to suppressSuggestions (both Groovy DSL and Kotlin DSL).

So if you use isAssertsEnabled with the Groovy DSL in v0.2, be prepared for a breaking change in the upcoming v0.3.

from gradle-nullaway-plugin.

Yep I was super lazy and only checked the README (and am using Groovy so didn't get autocomplete). Thanks!

from gradle-nullaway-plugin.

Hmm can't seem to make this work in the Groovy DSL with 0.2 actually. Tried isAssertsEnabled = true, isAssertsEnabled = Boolean.TRUE, isAssertsEnabled.set(true), and assertsEnabled = true. Everything throws unknown property exception.

from gradle-nullaway-plugin.

😭

Will have to add a proper test using the Groovy DSL then, like I did in the errorprone plugin (tbroyer/gradle-errorprone-plugin@39e8350)

from gradle-nullaway-plugin.

With 0.2, that should work with isAssertsEnabled().set(true)

from gradle-nullaway-plugin.

Yep, that works. Thanks!

JakeWharton/dagger-reflect#123

from gradle-nullaway-plugin.

Fwiw, I just released v0.3 with the breaking changes.

from gradle-nullaway-plugin.