mobile-app icon indicating copy to clipboard operation
mobile-app copied to clipboard

Upgrading AGP version and Gradle version of app build files

Open hardik17771 opened this issue 5 months ago • 10 comments

Describe the bug

  • The app uses an old version of gradle and AGP causing it to give warning during build and also can cause issues later if not upgraded.
  • settings.gradle / build.gradle config files need upgradation.

hardik17771 avatar Aug 10 '25 12:08 hardik17771

hey @hardik17771 , is the issue resolved ?

ShashwatXD avatar Aug 27 '25 20:08 ShashwatXD

Hi @ShashwatXD , not yet because we'll be solving it in a later release. Till then, you can try solving it and submit a PR for the same.

hardik17771 avatar Aug 28 '25 09:08 hardik17771

hello again @hardik17771 , please check pr #431 , i have mentioned all the stuff i did to upgrade the agp.

ShashwatXD avatar Aug 28 '25 12:08 ShashwatXD

Hi @hardik17771 , can you please take a look at my PR #431.

ShashwatXD avatar Aug 30 '25 11:08 ShashwatXD

Hi @ShashwatXD , please check why the PR is failing tests, likely due to commit convention. LMK when that is resolved

hardik17771 avatar Aug 30 '25 16:08 hardik17771

Hi @hardik17771 , i went through the failing tests and have fixed the build as well as the commit convention,

  • gradle/wrapper/gradle-wrapper.properties
  • build.gradle
  • settings.gradle
  • android/app/build.gradle
  • pubspec.yaml

The PR shows more files because of how I squashed the branch history, but only the above files are relevant.

Could you please re-run the checks

ShashwatXD avatar Aug 30 '25 18:08 ShashwatXD

@hardik17771 PR is ready for review :))

ShashwatXD avatar Aug 31 '25 16:08 ShashwatXD

Sure, give me some time to review.

hardik17771 avatar Aug 31 '25 17:08 hardik17771

@hardik17771 just checked the latest master, the build is already upgraded.

ShashwatXD avatar Sep 15 '25 09:09 ShashwatXD

No issues, it got covered in last merged PR. You can create any new issue and work for on the same. Thanks!

hardik17771 avatar Sep 15 '25 09:09 hardik17771