@marandaneto @davidmartos96 Maybe an integration test to check that the plugin builds with Gradle 7 and 8 would make sense to add?

@krystofwoldrich Is there any test of that kind in the repo? I'm not sure how it would work or if it would be worth the effort.

This change is equivalent to the one the official plugins have recently pushed in latest versions.
Like here: https://pub.dev/packages/shared_preferences_android/changelog
Here is the commit: flutter/packages@a86beaf
The if statement is for Gradle versions not compatible with the namespace attribute. It should build fine for different Gradle versions.

@marandaneto @davidmartos96 Maybe an integration test to check that the plugin builds with Gradle 7 and 8 would make sense to add?

Since we don't use AGP 8 yet, this is an integration test per see :)
Also every flutter package does this already flutter/packages@a86beaf
Its safe to go, otherwise we'd need to set up another sample app just for that (CI overhead, etc), but I'd do if the change was more fragile.

Thanks @davidmartos96

@davidmartos96 can you add a changelog entry? https://github.com/getsentry/sentry-dart/blob/main/CHANGELOG.md#fixes

@marandaneto Done!

Codecov Report

Patch and project coverage have no change.

Comparison is base (0006698) 90.23% compared to head (08a79dc) 90.23%.

@@           Coverage Diff           @@
##             main    #1427   +/-   ##
=======================================
  Coverage   90.23%   90.23%           
=======================================
  Files         181      181           
  Lines        5788     5788           
=======================================
  Hits         5223     5223           
  Misses        565      565           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.