Skip to content
This repository has been archived by the owner on Aug 30, 2023. It is now read-only.

fix desugaring Gson/AGP #177

Merged
merged 1 commit into from
Dec 1, 2019
Merged

fix desugaring Gson/AGP #177

merged 1 commit into from
Dec 1, 2019

Conversation

marandaneto
Copy link
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

make gson retro compatible with older versions of AGP

💡 Motivation and Context

when compiling a project and using the latest version of sentry-android, an error was happening due to:
google/gson#1597

investigation due to:
#171

💚 How did you test it?

compiling a sample project with all the AGP versions >= 3 till the latest one.

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • All tests passing

🔮 Next steps

find a way to compile a sample project with all AGP versions, to be sure that we don't release something which is breaking the build for older versions.

@codecov-io
Copy link

codecov-io commented Nov 30, 2019

Codecov Report

Merging #177 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #177   +/-   ##
=========================================
  Coverage     57.73%   57.73%           
  Complexity      526      526           
=========================================
  Files            72       72           
  Lines          2522     2522           
  Branches        223      223           
=========================================
  Hits           1456     1456           
  Misses          958      958           
  Partials        108      108

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4cf2bc0...f70a1bb. Read the comment docs.

@marandaneto marandaneto changed the title fix desugaring for gson fix desugaring Gson/AGP Nov 30, 2019
@marandaneto marandaneto merged commit d56d805 into master Dec 1, 2019
@marandaneto marandaneto deleted the fix/desugaring branch December 1, 2019 13:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants