Always keep fields of annotated enums

[technoir42]

We need to keep enum fields for EnumJsonAdapter even if they are not used elsewhere.

@JsonClass(generateAdapter = false)
enum class StatusCode {
    @Json(name = "success")
    Success,
    @Json(name = "error")
    Error
}

if (status == StatusCode.Success) {
    ...
}

-keepclassmembernames won't prevent StatusCode.Error from being removed that's why -keepclassmembers should be used instead.
@gabrielittner

[ZacSweers]

errrr I don't think you should be annotating enums with @JsonClass, you should just keep enums (or just the ones you specifically need) instead. I'm not sure the generated adapter for this would even work, let alone be something that Moshi would look for in its adapter resolution.

[technoir42]

@hzsweers ProGuard rule for enums annotated with @JsonClass was added a day ago. See #689 and #691 for context.

[ZacSweers]

I've been on vacation, so I missed that :). Just noticed the = false bit as well, so in that case I think this looks good 👍

[gabrielittner]

LGTM.

I didn't run into this but I think that's just because my enums are used in exhaustive when statements so all fields are used.

[adams-y-chen]

My enum class name is still obfuscated in the apk after added the @JsonClass(generateAdapter = false). I'm using moshi 1.12.0.
Do I have to add proguard rule to my app project in addition to annotate my enum class?

@JsonClass(generateAdapter = false)
enum class StatusCode {
    @Json(name = "success")
    Success,
    @Json(name = "error")
    Error
}

Updates:
Notice that my AGP version is 4.2.0. Do I have to upgrade AGP to new version to use support this annotation?