search

Kantis / ks3

KotlinX Serialization Standard Serializers (KS3)
Apache License 2.0
19 stars 4 forks source link

Kantis/enable ktlint #101

Closed Kantis closed 7 months ago

Kantis commented 7 months ago

@aSemy any opinions on adding a linter?

aSemy commented 7 months ago

Personally I loathe them 😅. When I develop I prefer being able to hack-around and run tests before coming back to tidy it up later, and tools like ktlint interrupt my flow because throw errors during compilation. I prefer committing the IDE styles (e.g. https://github.com/adamko-dev/dokkatoo/tree/v2.0.0/.idea/codeStyles).

But you commit to ks3 more than me, so if it helps you, then go for it :) I'll figure out how to manually disable ktlint if needs be.

Kantis commented 7 months ago

You can get ktlint to run and format automatically on save, so it shouldnt get in the way.

I'll merge, but feel free to complain if it gets too tedious and we can reconsider.