-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updating auth firebase google services #530
base: master
Are you sure you want to change the base?
Updating auth firebase google services #530
Conversation
Hello @jbors this worked from my side and all UI Tests passed locally, let's see if it works in bitrise. |
No, it's not fixed, the error comes between the virtual machine of bitrise and R8 version and Kotlin compatibility, do you know if we made any changes related to that? @jbors "WARNING: D8: An error occurred when parsing kotlin metadata. This normally happens when using a newer version of kotlin than the kotlin version released when this version of R8 was created. To find compatible kotlin versions, please see: https://developer.android.com/studio/build/kotlin-d8-r8-versions |
I mean, yeah we bumped kotlin versions and I have been messing with proguard rules. Doesn't explain why it's only when we bump this package that it goes wrong. Maybe it matters which version of kotlin the package has been compiled on? |
Yes probably, I tried lowering kotlin version, the problem is between bitrise virtual machine app and kotlin version, maybe they are behind and there is some incompatibility. I think the VM job passed, if everything passes let's merge it, and update to kotlin 2.0 when it's stable. |
I mean, alternatively we just do not update this for now and wait for kotlin version to catch up? |
Kotlin 2.0 is already out there but for now it only works for Android Studio canary versions, I think it's better to lower the kotlin version for now so it doesn't block us from updating anything else. El 22 jun 2024 10:28, Joris Borsboom ***@***.***> escribió:
I mean, alternatively we just do not update this for now and wait for kotlin version to catch up?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>
|
@jbors Let's keep this branch/PR open to experiment when we can update few more things, it looks like it still fails. It's best to retry again when we update to kotlin 2.0 |
@simtop Cool. Thanks for helping out! |
Updating google play services auth and firebase bom and google services plugin.