Looks like release build of app is crashing on sta...

# confetti

[John O'Reilly]

Looks like release build of app is crashing on startup......can trace this back to following commit.....still looking in to it but in case anyone might know cause https://github.com/joreilly/Confetti/commit/d6a6ead638f619b8e8eb916a3b797af8e99db5a1#diff-7cdef30b2089db76c43a331c1[…]3bc5d8f74a90f0fbf5648deb59da86

[John O'Reilly]

@yschimke I tried pulling in latest horologist alpha but getting issue using that too

[John O'Reilly]

maybe related to following

// <https://github.com/firebase/firebase-android-sdk/issues/5997>
        exclude(group = "com.google.firebase", module = "protolite-well-known-types")

[yschimke]

Oooffff

[yschimke]

Feel free to revert that and I'll fix.

[John O'Reilly]

I went back to latest code and reverted to following but still crashing for some reason

horologist = "0.6.12"

[yschimke]

I can take a look in a couple of hours

[John O'Reilly]

no rush at all

[yschimke]

Ok, will do.

[yschimke]

And sorry

[John O'Reilly]

the public version of app is fine....only noticed as I thought I should publish new version to internal test channel

[John O'Reilly]

I probably should have some kind of test that uses the release version

[John O'Reilly]

actually, I forgot to comment out exclude(group = "com.google.firebase", module = "protolite-well-known-types") when reverting.....able to run release build now with that

[John O'Reilly]

could also be that we just need to add something to proguard config

[yschimke]

Thanks, and sorry you had to fix it. I'll add back with a release test that reproduces the issue and then the fix

[John O'Reilly]

no worries at all.....I'm happy project is good playground to find issues like this

[John O'Reilly]

getting following error with that

e: file:///home/runner/work/Confetti/Confetti/wearApp/src/test/kotlin/dev/johnoreilly/confetti/wear/surfaces/TileScreenshotTest.kt:48:13 Unresolved reference 'TileLayoutPreview'.

[John O'Reilly]

I can merge this as is if you want to follow up with changes for that?

[yschimke]

Can you comment out that section? Or delete the test?

[John O'Reilly]

yeah, I can do that

[John O'Reilly]

also just tried following and seems to work around the issue

-keep class com.google.firebase.** { *; }

[John O'Reilly]

(with still using that updated horologist version)