Should we use trailing commas in Kotlin code?

[mnonnenmacher]

In frontend TypeScript code we use trailing commas, but we do not use them in Kotlin code. This can be a bit confusing when working on both parts of the code.

IMO the main benefit of trailing commas is that it reduces the diff when adding an item to the end of a list.

Please share if you would like to enable trailing commas in Kotlin code as well or prefer to keep it as is. If the vote is positive in a week, I will configure the respective Detekt rules accordingly.

[sschuberth]

FYI, these are the respective Ktlint rules that Detekt uses.

[lamppu]

Btw, this would also be easy to change for the UI code if wanted, just set "trailingComma": "none" in .prettierrc 1 and run pnpm format.

[mnonnenmacher]

I like trailing commas so from my side there is no need to change it in the UI code even if the vote for Kotlin is negative, but that's up to @eclipse-apoapsis/frontend-devs to decide.

[mnonnenmacher]

As there was no majority for enabling trailing commas, we will keep the current configuration.
(I didn't know the GitHub UI shows the first option as selected in case of a tie.)