Skip to content
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

Add common proguard rules #141

Merged
merged 1 commit into from
May 6, 2024
Merged

Add common proguard rules #141

merged 1 commit into from
May 6, 2024

Conversation

emilypgoogle
Copy link
Collaborator

When we migrated to common, our types got moved and the proguard rule no longer applied. I think this should be adequate to handle all of our serialized types, but if there are more outside of common that'll also need to be kept.

@emilypgoogle emilypgoogle requested a review from rlazo May 6, 2024 20:41
@rlazo rlazo merged commit c2512bc into main May 6, 2024
5 checks passed
@rlazo rlazo deleted the ep/common-proguard branch May 6, 2024 23:24
PatilShreyas pushed a commit to PatilShreyas/generative-ai-kmp that referenced this pull request Sep 21, 2024
When we migrated to common, our types got moved and the proguard rule no
longer applied. I think this should be adequate to handle all of our
serialized types, but if there are more outside of common that'll also
need to be kept.
PatilShreyas pushed a commit to PatilShreyas/generative-ai-kmp that referenced this pull request Sep 21, 2024
When we migrated to common, our types got moved and the proguard rule no
longer applied. I think this should be adequate to handle all of our
serialized types, but if there are more outside of common that'll also
need to be kept.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants