Skip to content
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

Creates a base-application.gradle #448

Merged

Conversation

vegaro
Copy link
Contributor

@vegaro vegaro commented Jan 19, 2022

Created a base-application.gradle to remove duplicated configuration in the applications we have in the repository.

Suggested by @NachoSoto in #445 (comment)

@shortcut-integration
Copy link

This pull request has been linked to Shortcut Story #12565: Create base gradle file for applications.

@vegaro vegaro requested a review from a team January 19, 2022 12:58
Copy link
Contributor

@NachoSoto NachoSoto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yay 🥳


android {
compileSdkVersion 29
buildToolsVersion "30.0.3"

defaultConfig {
applicationId "com.revenuecat.purchases_sample"
minSdkVersion 26
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this a diff minSdk for a reason?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know, I think it has a dependency that requires 26, but I am not sure

defaultConfig {
applicationId "com.revenuecat.purchases_sample"
minSdkVersion 26
targetSdkVersion 29
targetSdkVersion compileVersion
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can just remove this now right?

Copy link
Contributor

@beylmk beylmk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥳 this is AWESOME. minor comments but they could be a follow-up PR

@vegaro vegaro merged commit dd5b14e into main Jan 20, 2022
@vegaro vegaro deleted the cesardelavega/sc-12565/create-base-gradle-file-for-applications branch January 20, 2022 21:43
@vegaro vegaro mentioned this pull request Feb 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants