-
Notifications
You must be signed in to change notification settings - Fork 177
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
Use includeBuilds for better local development #65
Conversation
example/android/settings.gradle
Outdated
|
||
// Run enableLocalBuild task to enable building purchases-hybrid-common from your local copy | ||
if (file(".composite-enable").exists()) { | ||
includeBuild ('../../../../purchases-hybrid-common/android') |
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.
is this line assuming the local path of purchases-hybrid-common?
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.
yes, what could we do to make this easier? maybe an environment variable?
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.
env variable would be the absolute ideal
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.
let me try that
@aboedo I changed so the task now accepts a parameter with the path:
|
awesome!! |
* creates task to enable local builds * upgrades versions * removes purchases dependency * adds parameter to specify path
Inspired by https://proandroiddev.com/saying-goodbye-to-snapshots-with-gradles-composite-builds-bc98751392f6 and https://galex.co.il/2017/09/12/The_Best_of_Two_worlds_with_Gradle_Composite_Builds.html.
This adds two new gradle tasks
enableLocalBuild
anddisableLocalBuild
. By runningenableLocalBuild
gradle will use the local copy ofpurchases-hybrid-common
instead of the one dowloaded from maven.Depends on github.com/RevenueCat/purchases-hybrid-common/pull/44