I cannot figure out a way to format `private val styleSource: NullForUndefined<@Serializable(with...

# ktlint

[Michael Krussel]

I cannot figure out a way to format private val styleSource: NullForUndefined<@Serializable(with = LayerSourceSerializer::class) SourceDto> = NullForUndefined(null) to make the AnnotationOnSeparateLine rule happy.

I'm using ktlint from detekt.

[Vampire]

Just let ktlint fix and not complain and you see it.
But as the rule says, it must be on a separate line:

private val styleSource: NullForUndefined<
    @Serializable(with = LayerSourceSerializer::class)
    SourceDto
    > = NullForUndefined(null) to make the AnnotationOnSeparateLine

[Michael Krussel]

I still get a two warnings about AnnotationOnSeparateLine with that format. One after the < and one at the >.

[David Meyer]

Yuck! There's an issue with the indentation rule too, since the opening and closing angle brackets should not be indented differently.

It's no secret that I am not a fan of this annotation rule in its current state. So my advice is to disable it until it can be configured to format the code in a more readable way.

You may also want to ask yourself why you are putting complicated annotations like this on generic type arguments. There is probably a better way.

[Michael Krussel]

thanks

[phldavies]

private typealias LayerSource =
    @Serializable(with = LayerSourceSerializer::class)
    SourceDto
private val styleSource: NullForUndefined<LayerSource> = NullForUndefined(null)

[phldavies]

I had to do similar to get around this ugly mess:

@Serializable
private data class BigDecimalWithBytes(
    val unscaledValue: @Serializable(with = LongOrBytesBigIntegerSerializer::class) BigInteger,
    val scale: Int,
)

ended up as

@Serializable
private data class BigDecimalWithBytes(
    val unscaledValue:
    @Serializable(with = LongOrBytesBigIntegerSerializer::class)
    BigInteger,
    val scale: Int,
)

[Michael Krussel]

Interesting, thanks for the information.