This should be a easy and quick task for the LLM but a annoying task for a human.
Currently the Kotlin compiler is reporting many lines of code in Columba because some years ago the recommended code style for Kotlin have been set to something different then the current style in Columba.
In general in Kotlin you should should avoid code style like this:
data class Something(
val someValue: String // comment about someValue
)
And instead use this:
data class Something(
// comment about someValue
val someValue: String
)
There are reasons why the Kotlin developers say the Kotlin code should look different then its currently on many places in Columba.
Some examples:
columba/app/src/test/java/network/columba/app/ui/model/CodecProfileTest.kt:30
columba/app/src/test/java/network/columba/app/ui/screens/SettingsScreenBannerVisibilityTest.kt:202
columba/app/src/test/java/network/columba/app/ui/util/MarkerDeclutterTest.kt:262
and many, many more places in the Columba code.
The compiler currently throws on such lines this: A comment in a 'value_argument_list' is only allowed when placed on a separate line (cannot be auto-corrected)
It should be a really simple task to say to the LLM to optimize the code style to follow this Kotlin rule.
This should be a easy and quick task for the LLM but a annoying task for a human.
Currently the Kotlin compiler is reporting many lines of code in Columba because some years ago the recommended code style for Kotlin have been set to something different then the current style in Columba.
In general in Kotlin you should should avoid code style like this:
And instead use this:
There are reasons why the Kotlin developers say the Kotlin code should look different then its currently on many places in Columba.
Some examples:
columba/app/src/test/java/network/columba/app/ui/model/CodecProfileTest.kt:30
columba/app/src/test/java/network/columba/app/ui/screens/SettingsScreenBannerVisibilityTest.kt:202
columba/app/src/test/java/network/columba/app/ui/util/MarkerDeclutterTest.kt:262
and many, many more places in the Columba code.
The compiler currently throws on such lines this: A comment in a 'value_argument_list' is only allowed when placed on a separate line (cannot be auto-corrected)
It should be a really simple task to say to the LLM to optimize the code style to follow this Kotlin rule.