Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update kotlin version in examples #4844

Merged
merged 3 commits into from
May 23, 2024
Merged

Conversation

eymar
Copy link
Collaborator

@eymar eymar commented May 21, 2024

Changes:

Testing:

  • ./gradlew build in every example
  • run some examples in simulators (or desktop, browser)

@eymar eymar marked this pull request as ready for review May 22, 2024 09:22
@eymar eymar requested a review from igordmn May 22, 2024 09:23
@@ -1,7 +1,7 @@
kotlin.code.style=official
xcodeproj=./iosApp
android.useAndroidX=true
org.gradle.jvmargs=-Xmx3g
org.gradle.jvmargs=-Xmx8g
Copy link
Collaborator Author

@eymar eymar May 22, 2024

Choose a reason for hiding this comment

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

increased it here and in other places, becuase otherwise ./gradlew build was failing with OOM. Building the targets separately was fine though.

@eymar eymar requested a review from MatkovIvan May 22, 2024 09:26
@eymar eymar merged commit 4c01cbd into master May 23, 2024
@eymar eymar deleted the ok/update_kotlin_in_examples branch May 23, 2024 08:23
eymar added a commit that referenced this pull request May 29, 2024
Changes:
- update all examples to Kotlin 2.0
- migrate to using kotlin("plugin.compose") for Compose compiler
configuration
- remove deprecated Compose Web gradle API usages

- added a workaround for
#4848 in
Jetsnack example

Testing:
- `./gradlew build` in every example
- run some examples in simulators (or desktop, browser)

(cherry picked from commit 4c01cbd)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants