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

Update consumer-rules.pro #1040

Merged
merged 2 commits into from May 13, 2020
Merged

Update consumer-rules.pro #1040

merged 2 commits into from May 13, 2020

Conversation

MariusVolkhart
Copy link
Contributor

GSON's TypeAdapter is an abstract class, not an interface.

GSON's TypeAdapter is an abstract class, not an interface.
@iambmelt
Copy link
Member

The proguard config that we ship has the verbatim guidance from google's repo:
https://github.com/google/gson/blob/master/examples/android-proguard-example/proguard.cfg#L18

If you believe there's an issue with this config, I'd try to submitting an issue over there -- if that merges, we can take it here

@iambmelt iambmelt self-assigned this May 11, 2020
@iambmelt iambmelt added the More Info Additional information is required to diagnose, troubleshoot, or confirm label May 11, 2020
@MariusVolkhart
Copy link
Contributor Author

Done. See google/gson#1693

@iambmelt iambmelt added Bug - P2 A problem that needs to be fixed for a feature to function as intended Issue Triage The engineering team has looked into the issue, understood the issue, labelled/classified the issue and removed More Info Additional information is required to diagnose, troubleshoot, or confirm labels May 13, 2020
@iambmelt
Copy link
Member

@MariusVolkhart Brilliant. Thanks

I'll take the liberty of updating this branch with the latest changes for a clean merge. Thank you for your contribution!

Copy link
Contributor

@shahzaibj shahzaibj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@iambmelt iambmelt merged commit e2c7b13 into AzureAD:dev May 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug - P2 A problem that needs to be fixed for a feature to function as intended Issue Triage The engineering team has looked into the issue, understood the issue, labelled/classified the issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants