-
Notifications
You must be signed in to change notification settings - Fork 9.8k
[all_plugins] Compile all plugins together #1776
Conversation
package="io.plugins.all_plugins" | ||
xmlns:tools="http://schemas.android.com/tools"> | ||
|
||
<uses-sdk tools:overrideLibrary="io.flutter.plugins.camera"/> |
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.
Required because camera plugin is 21+
REPO_DIR="$(dirname "$SCRIPT_DIR")" | ||
|
||
source "$SCRIPT_DIR/common.sh" | ||
check_changed_packages > /dev/null | ||
|
||
cd $REPO_DIR/examples/all_plugins | ||
flutter clean > /dev/null | ||
(cd "$REPO_DIR" && pub global run flutter_plugin_tools gen-pubspec --exclude firebase_core,firebase_ml_vision) |
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.
Exclude firebase_core
because all the other firebase plugins depend on it. This caused a problem when compiling.
Exclude firebase_ml_vision
because it requires iOS 9.0+ which caused a problem when compiling with google_maps_flutter
. I didn't want to change either plugin in this PR.
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.
sgtm. note that we'll be moving these plugins out of this repo soon.
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.
minor issue, lgtm otherwise
REPO_DIR="$(dirname "$SCRIPT_DIR")" | ||
|
||
source "$SCRIPT_DIR/common.sh" | ||
check_changed_packages > /dev/null | ||
|
||
cd $REPO_DIR/examples/all_plugins | ||
flutter clean > /dev/null | ||
(cd "$REPO_DIR" && pub global run flutter_plugin_tools gen-pubspec --exclude firebase_core,firebase_ml_vision) |
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.
sgtm. note that we'll be moving these plugins out of this repo soon.
Description
Add pubspec generating tool to
build_all_plugins_app.sh
script. This will add all the plugins in the repo to the all_plugins app and test compilation.