-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Fix Android flavors build.gradle entry example #8177
Conversation
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.
@parlough : Could use updates on the flavorDimension
options.
Thanks @atsansone! Please take another look :) |
flavorDimensions
and use correct propertyThere 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.
lgtm!
This looks ready to land! Thanks, @parlough! |
Yes, please, @parlough. Remember, it's "progress, not perfection!" :D |
Please do file an issue for the stuff that needs further tweaking and land it. |
I've updated #7979 to track those suggestions and other necessary changes. |
@parlough, excellent! @atsansone, can you land this? |
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.
@parlough : Concur. Go ahead and merge.
This section could use some further updates, but this at least fixes the existing errors that show up.
Fixes #8096