Skip to content

fix(android): Integer conversion of compileSdk/minSdk/targetSdk#677

Closed
saschpe wants to merge 1 commit intoionic-team:mainfrom
saschpe:android-version-custom
Closed

fix(android): Integer conversion of compileSdk/minSdk/targetSdk#677
saschpe wants to merge 1 commit intoionic-team:mainfrom
saschpe:android-version-custom

Conversation

@saschpe
Copy link
Copy Markdown

@saschpe saschpe commented Nov 2, 2021

Gradle properties are always parsed as strings from gradle.properties.
While Gradle Groovy can utilize the ext Gradle plugin directly, Gradle
Kotlin DSL projects have to specify properties in gradle.properties.
Convert those to integers as required by Android Gradle Plugin (AGP).
This won't affect projects that still use Gradle Groovy.

Resolves #ionic-team/capacitor#5194

@saschpe saschpe force-pushed the android-version-custom branch 4 times, most recently from 8ad1030 to 68e7edb Compare November 9, 2021 09:17
@saschpe saschpe force-pushed the android-version-custom branch 2 times, most recently from b8f8316 to a717e57 Compare November 19, 2021 09:48
@saschpe saschpe force-pushed the android-version-custom branch from a717e57 to bfce275 Compare December 7, 2021 08:04
@saschpe saschpe force-pushed the android-version-custom branch from bfce275 to 6680673 Compare December 14, 2021 11:21
@saschpe saschpe force-pushed the android-version-custom branch from 6680673 to a13e54b Compare June 20, 2022 08:01
@saschpe saschpe force-pushed the android-version-custom branch from a13e54b to 30235bc Compare March 13, 2023 13:06
@saschpe saschpe force-pushed the android-version-custom branch from 30235bc to a25529e Compare March 24, 2023 07:55
@saschpe saschpe force-pushed the android-version-custom branch from a25529e to 4655c7c Compare March 28, 2023 07:12
@saschpe saschpe force-pushed the android-version-custom branch from 4655c7c to bee9d41 Compare April 5, 2023 10:47
Gradle properties are always parsed as strings from `gradle.properties`.
While Gradle Groovy can utilize the `ext` Gradle plugin directly, Gradle
Kotlin DSL projects have to specify properties in `gradle.properties`.
Convert those to integers as required by Android Gradle Plugin (AGP).
This won't affect projects that still use Gradle Groovy.

Resolves #ionic-team/capacitor#5194
@saschpe saschpe force-pushed the android-version-custom branch from bee9d41 to e22cfda Compare August 18, 2023 11:33
@saschpe
Copy link
Copy Markdown
Author

saschpe commented Sep 12, 2024

Final ping, I think this is still relevant. Worth rebasing once more?

@saschpe saschpe closed this Sep 12, 2024
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.

1 participant