Nivaldo Bondança

Results 124 comments of Nivaldo Bondança

This seems like a bit of an [edge case](https://blog.gradle.org/simpler-kotlin-dsl-property-assignment#:~:text=option%20to%20overload%20the%20%3D%20assignment%20operator%20for%20final%20properties&text=written%20as%20a%20Kotlin%20compiler%20plugin), but I don't see why we could not add this exception to the [list here]([https://github.com/facebook/ktfmt/blob/58777b2e386128f15042de5c2698511ef0771352/co[…]/main/java/com/facebook/ktfmt/format/RedundantImportDetector.kt](https://github.com/facebook/ktfmt/blob/58777b2e386128f15042de5c2698511ef0771352/core/src/main/java/com/facebook/ktfmt/format/RedundantImportDetector.kt#L34-L79)). Feel free to submit a PR

@greyhairredbear do you know if this is related to #420?

I tried to format you code here and it fails due to the `@interface` there, as that's not valid Kotlin. Could you revisit and share a piece of code where...

What are you suggesting to be done here? Change the default to 4 spaces? You could specify `--kotlinlang-style` to use the Kotlin recommended one. We do not have the intention...

What styles did you have in mind to add this feature to @hongping-quora?

This is part of the 1.0 proposal (https://github.com/facebook/ktfmt/discussions/451#discussioncomment-9305741)

Sorry about the delay on the response to #420, I hope we can close it this week 🙂

If you want, you could update those yourself @atulgpt, we'd be happy to merge your Pull Request

@atulgpt they LGTM. If you could remove any visual artifacts such as the red symbols and red squiggly line, that would be great 😃

cc @cgrushko in case you have strong thoughts about this