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

Update plugin kotlin-multiplatform to v1.6.20 #296

Merged
merged 5 commits into from
Apr 9, 2022

Conversation

juul-mobile-bot
Copy link
Contributor

@juul-mobile-bot juul-mobile-bot commented Apr 2, 2022

This PR contains the following updates:

Package Type Update Change
org.jetbrains.kotlin.multiplatform plugin patch 1.6.10 -> 1.6.20

Configuration

📅 Schedule: At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

@juul-mobile-bot juul-mobile-bot requested a review from a team April 2, 2022 09:09
@juul-mobile-bot juul-mobile-bot requested a review from twyatt as a code owner April 2, 2022 09:09
@juul-mobile-bot juul-mobile-bot requested a review from sdonn3 April 2, 2022 09:09
@twyatt
Copy link
Member

twyatt commented Apr 3, 2022

Note to self: prior to merge, test compilation on Apple silicon.

@twyatt twyatt force-pushed the renovate/org.jetbrains.kotlin.multiplatform-1.x branch from 8d15f36 to 9f94450 Compare April 3, 2022 23:09
Comment on lines -1 to -3
# https://kotlinlang.org/docs/mpp-share-on-platforms.html
kotlin.mpp.enableGranularSourceSetsMetadata=true
kotlin.native.enableDependencyPropagation=false
Copy link
Member

Choose a reason for hiding this comment

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

These removed options are deprecated in Kotlin 1.6.20 per release announcement.

Comment on lines -39 to -43
// Workaround for https://youtrack.jetbrains.com/issue/KT-49109 that allows building on Apple Silicon
// This should no longer be needed with Kotlin version 1.6.20
rootProject.plugins.withType<org.jetbrains.kotlin.gradle.targets.js.nodejs.NodeJsRootPlugin> {
rootProject.the<org.jetbrains.kotlin.gradle.targets.js.nodejs.NodeJsRootExtension>().nodeVersion = "16.0.0"
}
Copy link
Member

Choose a reason for hiding this comment

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

Node.js 16.x.x should be default for Kotlin 1.6.20, so this workaround should no longer be needed.

@sdonn3 sdonn3 enabled auto-merge (squash) April 4, 2022 23:06
@twyatt twyatt requested a review from QuantumRand April 8, 2022 23:33
@twyatt
Copy link
Member

twyatt commented Apr 8, 2022

@QuantumRand can you confirm that this builds on Apple silicon?

./gradlew clean assemble

@QuantumRand
Copy link
Contributor

@QuantumRand can you confirm that this builds on Apple silicon?

./gradlew clean assemble

No issues building on M1 🚀

@twyatt twyatt disabled auto-merge April 9, 2022 05:07
@twyatt twyatt merged commit daa9f99 into main Apr 9, 2022
@twyatt twyatt deleted the renovate/org.jetbrains.kotlin.multiplatform-1.x branch April 9, 2022 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants