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

chore(kmm): decrease gradle agp version #87

Merged
merged 1 commit into from
May 2, 2023
Merged

Conversation

RyuNen344
Copy link
Contributor

@RyuNen344 RyuNen344 commented May 1, 2023

Why

#74

AGP 8.0 requests gradle 8, but gradle 8 can't assemble XCFramework
https://youtrack.jetbrains.com/issue/KT-56019

so, decrease gradle to 7.6.1

What

decrease gradle, agp version

@github-actions
Copy link

github-actions bot commented May 1, 2023

Dependency diff:

dependency diff
branch: chore/setup-ios -> chore/setup-kmm, module: app, configuration: releaseRuntimeClasspath
Upgraded Dependencies
org.jetbrains.kotlin:kotlin-stdlib:1.8.20, (changed from 1.8.10)
org.jetbrains.kotlin:kotlin-stdlib-common:1.8.20, (changed from 1.8.10)
org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.8.20, (changed from 1.8.10)
org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.8.20, (changed from 1.8.10)

@@ -14,12 +14,6 @@ java {
targetCompatibility = JavaVersion.VERSION_11
}

kotlin {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

jvmToolchain is a feature of gradle 8

@@ -26,6 +26,7 @@ hiltGradlePlugin = { group = "com.google.dagger", name = "hilt-android-gradle-pl

androidDesugarJdkLibs = { group = "com.android.tools", name = "desugar_jdk_libs", version.ref = "androidDesugarJdkLibs" }

composeCompiler = { module = "androidx.compose.compiler:compiler", version.ref = "composeCompiler" }
Copy link
Contributor Author

@RyuNen344 RyuNen344 May 1, 2023

Choose a reason for hiding this comment

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

define module to be checked by Renovate

@github-actions
Copy link

github-actions bot commented May 1, 2023

Test Results

2 files  ±0  2 suites  ±0   0s ⏱️ ±0s
1 tests ±0  1 ✔️ ±0  0 💤 ±0  0 ±0 
2 runs  ±0  2 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 72eb6ec. ± Comparison against base commit 4aaf250.

♻️ This comment has been updated with latest results.

kotlin.mpp.enableCInteropCommonization=true
kotlin.mpp.stability.nowarn=true
kotlin.mpp.androidSourceSetLayoutVersion=2
Copy link
Contributor Author

Choose a reason for hiding this comment

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

kotlin1.8 brings new source set layout
https://kotlinlang.org/docs/whatsnew18.html#configuration-and-setup

androidTest -> androidUnitTest
androidAndroidTest -> androidInstrumentedTest

Copy link
Contributor Author

Choose a reason for hiding this comment

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

package meta has already moved to build.gradle.
manifest file became empty file, so I deleted.
https://developer.android.com/build/configure-app-module#set-namespace

@RyuNen344 RyuNen344 self-assigned this May 1, 2023
Copy link
Member

@takahirom takahirom left a comment

Choose a reason for hiding this comment

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

Looks great!

@RyuNen344 RyuNen344 changed the base branch from chore/setup-ios to main May 2, 2023 05:16
@RyuNen344 RyuNen344 merged commit 7cccb81 into main May 2, 2023
5 checks passed
@RyuNen344 RyuNen344 deleted the chore/setup-kmm branch May 2, 2023 05:21
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

2 participants