Skip to content

Conversation

faogustavo
Copy link
Contributor

  • Updated Kotlin to 2.2.10
  • Updated com.vanniktech.maven.publish to 0.34.0
  • Updated Gradle to latest stable
  • Updated to use default source-set hierarchy
  • Removed all APIS related to the old memory model that are deprecated level error now
  • Fixed all tests

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Updates Kotlin to version 2.2.10 and modernizes the project by removing deprecated old memory model APIs, updating build tools, and migrating to standard Kotlin/Native threading patterns.

  • Updated Kotlin to 2.2.10 and removed deprecated freezing/memory model APIs
  • Replaced time measurement utilities with modern TimeSource.Monotonic API
  • Simplified Gradle build configuration using default source-set hierarchy

Reviewed Changes

Copilot reviewed 19 out of 50 changed files in this pull request and generated no comments.

Show a summary per file
File Description
gradle.properties Updated Kotlin version from 1.9.20 to 2.2.10
gradle/wrapper/gradle-wrapper.properties Updated Gradle to 8.14.3
sqliter-driver/build.gradle.kts Updated maven publish plugin, simplified build config with default hierarchy
.github/workflows/ Updated CI workflows and deployment scripts
src/nativeCommonMain/kotlin/co/touchlab/sqliter/util/Functions.kt Removed deprecated freeze/memory model utility functions
src/nativeCommonMain/kotlin/co/touchlab/sqliter/concurrency/SingleThreadDatabaseConnection.kt Removed wrapper class that was only needed for old memory model
Multiple test and main source files Replaced deprecated time APIs and removed freeze() calls

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@faogustavo faogustavo changed the title Kotlin Version & Deployment Script Updated Kotlin Version & Deployment Script Updates Sep 3, 2025
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