Since <@U0BFDUP0E> is back care to give status upd...
# squarelibraries
t
Since @jw is back care to give status update for Moshi future ? 🙂
j
Are these correlated? What's ambiguous about it?
c
Wait that wasn’t a sign to move back to GSON? Shoot.
t
You and Zac where the only 2 active recently on that one, Zac moved on, you are back to Square so was hoping to have some visibility, there's many PR without even a comment from square, usually this means project is in bad future 😞
😞 2
j
The library was already mostly feature complete prior to the Kotlin work so not being active is a feature, not a bug
t
Then I'll ask about https://github.com/square/moshi/pull/1123 🙂 the gain was huge so I'm in prod with that + some other fixes like the R8 auto keeping metadata, if there's no way it's merged then I'll move my fork to something I can better better maintain without Maven.
j
I don't really work on Moshi except the low-level JSON parts for fun. I use kotlinx.serialization for projects. So you'll probably have to keep waiting.
t
I unfortunately need streaming for very very large lists and from your recent posts this is not yet the case for kotlinx 😞 Am I allowed to ping Jesse there or is it considered bad after only a month?
j
I can look at it
t
@jessewilson Thanks, I'll then also PR for https://github.com/square/moshi/issues/1115 that would be nice to have before Android Studio 4 final is out, as AGP 4 will trigger this huge APK size increase.
j
4 is out last week
t
Lol yes sorry since I was testing R8 not in 4 final I forget that 😛 Anyway does not change the issue 🙂
@jw asking here to avoid spam on github. by rules you mean adding the following? Can you confirm keepclassmembers is enough ?
Copy code
# Keep helper method to avoid R8 optimisation that would keep all Kotlin Metadata when unwanted
-keepclassmembers class com.squareup.moshi.internal.Util {
    private static java.lang.String getKotlinMetadataClassName();
}