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 Kotlin 1.9 (and friends) #1123

Merged
merged 3 commits into from
Oct 23, 2023
Merged

Conversation

steve-the-edwards
Copy link
Contributor

@steve-the-edwards steve-the-edwards commented Oct 20, 2023

Upgrade to Kotlin 1.9 and other dependencies - e.g. Compose compiler, ktlint, etc.

Use Kotlin/JS (IR) for the Kotlin/JS

Run udpated dependencyGuardBaseline and ktlint format

Removed ktlint_code_style = official as that is no longer an option.

Base automatically changed from rick/no_kotlin_dsl to main October 23, 2023 14:45
Force kotlin 1.9 in build logic so that while using plugins that default to what is already being used they won't leak in previous versions.
@steve-the-edwards steve-the-edwards force-pushed the sedwards/update-kotlin-1.9 branch 2 times, most recently from b392ed0 to 96b822a Compare October 23, 2023 15:37
@steve-the-edwards steve-the-edwards marked this pull request as ready for review October 23, 2023 17:06
@steve-the-edwards steve-the-edwards force-pushed the sedwards/update-kotlin-1.9 branch from 96b822a to f4b6ccb Compare October 23, 2023 19:17
@@ -1,3 +1,8 @@
plugins {
// Hardcoded as this is upstream of the version catalog. Keep this in sync with that.
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a comment there pointing back at this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, there wasn't but that's a good point. Adding now in gradle/libs.versions.toml:

# Keep this in sync with what is hard-coded in build-logic/settings.gradle.kts as that is upstream
# of loading the library versions from this file but should be the same.
kotlin = "1.9.10"

@steve-the-edwards steve-the-edwards force-pushed the sedwards/update-kotlin-1.9 branch from f4b6ccb to 3524e8f Compare October 23, 2023 19:38
@steve-the-edwards steve-the-edwards merged commit 6014073 into main Oct 23, 2023
31 checks passed
@steve-the-edwards steve-the-edwards deleted the sedwards/update-kotlin-1.9 branch October 23, 2023 20:40
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.

2 participants