AGP8 compatibility

[deckerst]

Main change is adding namespace for AGP8 compatibility.
Rest is upgrades so that the example can compile on a modern setup with recent Flutter/Gradle.

[Nimrodda]

Thanks for the PR. I left some comments.
Also have you verified the the example app works?

[Nimrodda]

Please keep the existing minimum

[deckerst]

Ok. By the way, is there a reason for this minimum?

[Nimrodda]

PR title says AGP 8 but I see 7 here :)

[deckerst]

AGP 8 compatibility is for consumer projects using AGP 8.
Here changing to 7 was not necessary for AGP 8 compatibility, but just part of general upgrades from ancient versions used in this repo.

[deckerst]

Also have you verified the the example app works?

Yes. And this fix for AGP8 has been used in many repos already.

[Nimrodda]

checks failed Because flutter_nsd_example depends on cupertino_icons >=1.0.6 which requires SDK version >=2.19.0 <4.0.0, version solving failed.. Let's address that before I merge this.

[deckerst]

Interesting. In my environment I use a recent Dart SDK which requires null safety, so I needed to bump cupertino_icons version to make it build. But your GitHub workflow uses an old Dart SDK (< 2.19.0). That Dart SDK version depends on the Flutter version you specified in env.FLUTTER_VERSION which is setup at the repo level. So my understanding is that you can bump that FLUTTER_VERSION. Otherwise, feel free to reject this PR.

[Nimrodda]

Interesting. In my environment I use a recent Dart SDK which requires null safety, so I needed to bump cupertino_icons version to make it build. But your GitHub workflow uses an old Dart SDK (< 2.19.0). That Dart SDK version depends on the Flutter version you specified in env.FLUTTER_VERSION which is setup at the repo level. So my understanding is that you can bump that FLUTTER_VERSION. Otherwise, feel free to reject this PR.

I use FVM for Flutter versioning. Could you please update it and we see if it fixes the issue?