Android Studio / IDEA plugin yields different results than running ktfmt about ktfmt HOT 11 OPEN

@gino-m did you also update the ktfmt-gradle to the newer version (0.18.0)? I suspect the intellij plugin is working correctly but your codebase was formatted incorrectly with the older Gradle plugin

from ktfmt.

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

Unfortunately, not really. My original thought was that this might be because of different ktfmt versions, but according to the original post in the gradle plugin, this doesn't seem to be the case.

I can try if #420 produces the expected results.

UPDATE: @hick209 of course, you are correct, the @interface is not valid Kotlin... @gino-m are you sure this is a .kt file you are referencing here?

PS: @gino-m I suppose it would be helpful to also include the versions used in this post.

from ktfmt.

Just FYI: When using #420, the code stays unchanged (regardless of the formatting) due to it not being valid Kotlin. After removing the @interface-portion, the output is the following:

/**
 * Annotates view models to indicate that a single instance should be shared by all fragments in an
 * activity. Only one instance of such view models will be created per activity by {@link
 * ViewModelFactory#get(Fragment, Class)}.
 */
@Retention(RetentionPolicy.RUNTIME) public SharedViewModel {}

from ktfmt.

@hick209 So sorry, was moving too fast. That was Java - added a Kotlin example .

@greyhairredbear The formatting in the above snippet was produced by Android Studio ktfmt plugin 1.1.0.47, the Gradle plugin com.ncorti.ktfmt.gradle version 0.17.0. Added these to the updated issue description and shared additional deltas in https://github.com/google/ground-android/pull/2412/files.

from ktfmt.

FYI: Just tried the new version of the plugin released last week and the code you mentioned

@Retention(AnnotationRetention.RUNTIME)
annotation class SharedViewModel

gets reformatted to the code produced by the gradle plugin

@Retention(AnnotationRetention.RUNTIME) annotation class SharedViewModel

Seems to me this is not an issue anymore - probably has been fixed with #420. @gino-m could you verify that also is the case with the code you've mentioned, so we can close this?

from ktfmt.

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

from ktfmt.

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 we could repro the issue?

from ktfmt.

@gino-m this might be related: cortinico/ktfmt-gradle#277

from ktfmt.

Confirmed, I upgraded to 1.1.0.49 and the outputs for this particular use case now match.

There are still some differences, which I've tried to capture here: https://github.com/google/ground-android/pull/2456/commits

The first run of the latest AS plugin across our codebase introduced many changes, most of which are not modified by running ktfmt via Gradle again. Subsequent commits demonstrate formatting which is changes by both entry points on each run, namely: google/ground-android@b9736a1

from ktfmt.

Good catch. I upgraded ktfmt-gradle to 0.18.0 and reformatted (google/ground-android@9dc9da5).

Reformatting with AS ktfmt 1.1.0.49 still yields several diffs (google/ground-android@e6a4974), but subsequent runs of ktfmt-gradle only modify three files (google/ground-android@ea60727). Two of these changes get undone when running the AS plugin again (google/ground-android@ebe230e).

At this point I would expect the diffs, if any, to be symmetrical, but running ktfmt-gradle again only modifies one file (google/ground-android@81371be). After that, subsequent runs fight over a single delta (google/ground-android@e26f8d9).

Edit: Pasting screenshot here for convenience. Before = ktfmt-gradle 0.180, After = AS ktfmt 1.1.0.49:

If that delta is resolved the formatting should be stable across both plugins after each is run several times.

from ktfmt.

What is the output of the CLI? I ran ktfmt --google-style Config.kt locally and got the second output - so it seems the plugin is behaving as expected.

from ktfmt.