-
-
Notifications
You must be signed in to change notification settings - Fork 871
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
refactor(package_info_plus): Migrate Android example to use the new plugins declaration #2715
Conversation
allprojects { | ||
repositories { | ||
google() | ||
mavenCentral() | ||
} | ||
} | ||
|
||
rootProject.buildDir = '../build' | ||
rootProject.layout.buildDirectory = '../build' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not related to the guidance that I shared, but something that was deprecated in Gradle 8.2.
https://docs.gradle.org/current/userguide/upgrading_version_8.html#project_builddir
Considering that we require 8.4 now due to Android Gradle Plugin 8.3.0, I thought that it won't hurt to address this deprecation as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reverted this change to stick with only changes from the guidance I shared. Hope it works now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like after reverting this change all works as expected.
Hmm, it is strange that such error happens as it works fine locally 🤔 |
a130a6f
to
3491dad
Compare
a4c02e7
to
e7f2f45
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice cleanup!
Description
Considering that fact that some of Plug Plugins will require Flutter 3.19 as min version in their next release I decided to also address the deprecation in how we declare Android related plugins in example apps. Currently users would see warnings during example app builds like this one: https://github.com/fluttercommunity/plus_plugins/actions/runs/8308735389/job/22743183649#step:5:96
There is also an official guidance from the Flutter team on this topic: https://docs.flutter.dev/release/breaking-changes/flutter-gradle-plugin-apply
With this change our example apps will be more or less up to date and show a good example on how to apply the new approach to apps.
Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?
!
in the title as explained in Conventional Commits).