Support for custom build types on Android #11354
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jakobr-google
approved these changes
Jul 31, 2017
Contributor
jakobr-google
left a comment
jakobr-google
left a comment
There was a problem hiding this comment.
LGTM, +1, would review again.
| * | ||
| * Note: The BuildType DSL type is not public, and is therefore just Object here. | ||
| */ | ||
| private void addFlutterJarCompileDependency(Project project, Object buildType) { |
Contributor
There was a problem hiding this comment.
I believe Groovy-style is to leave the type out when you can't/don't want to specify it for arguments (and use def when declaring untyped variables). Section 3 of http://groovy-lang.org/style-guide.html. :)
Hixie
added a commit
that referenced
this pull request
Aug 1, 2017
* Revert "Make plugins add their repos to projects in the consuming app (#11447)" This reverts commit abe1e25. * Revert "Support for custom build types on Android (#11354)" This reverts commit 87eec71. * Revert "add a profile() method (#11443)" This reverts commit 561d17a. * Revert "Fix documentation based on dartdoc's warnings (#11428)" This reverts commit 6655074. * Revert "Improve some docs around WillPopScope. (#11429)" This reverts commit 58a28a2. * Revert "temporarily disable broken driver test in integration_ui (#11440)" This reverts commit 764515e. * Revert "style fix" This reverts commit 00bfc86. * Revert "tests for waitFor/waitForAbsent" This reverts commit 31d2ee9. * Revert "Always evaluate the finder in `driver.waitFor()` and `driver.waitForAbsent()`" This reverts commit 11d7c79.
Contributor
|
This patch noticeably affected the build_time_millis benchmark. |
This was referenced Aug 2, 2017
|
How do you run these buildTypes from the Flutter CLI? |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
5 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Custom build variants are now supported by the Flutter Gradle plugin. This involves making sure that Flutter dependencies are injected into Gradle configurations and tasks beyond the default Android
debugandreleasebuild types.Fixes #11095