Bump to 17 and fix formatting

Signed-off-by: mramotar <[email protected]>

Author: matt-ramotar

.github/workflows/.ci_test_and_publish.yml

@@ -20,7 +20,7 @@ jobs:
         uses: actions/setup-java@v2
         with:
           distribution: zulu
-          java-version: 11
+          java-version: 17
 
       - name: Upload Artifacts
         run: ./gradlew publishAllPublicationsToMavenCentralRepository --no-daemon --no-parallel
@@ -56,7 +56,7 @@ jobs:
         uses: actions/setup-java@v2
         with:
           distribution: zulu
-          java-version: 11
+          java-version: 17
       - name: Run tests
         run: ./gradlew check --rerun-tasks --stacktrace
       - name: Upload code coverage

.gitignore

@@ -49,4 +49,6 @@ captures/
 
 store/kover
 *.podspec
-yarn.lock
+yarn.lock
+
+*/kover

build.gradle.kts

@@ -1,5 +1,5 @@
 plugins {
-    id("org.jlleitschuh.gradle.ktlint") version "11.0.0"
+    id("org.jlleitschuh.gradle.ktlint") version "12.1.0"
     id("com.diffplug.spotless") version "6.4.1"
 }
 
@@ -49,13 +49,13 @@ subprojects {
 tasks {
     withType<org.jetbrains.kotlin.gradle.tasks.KotlinCompile> {
         kotlinOptions {
-            jvmTarget = "11"
+            jvmTarget = "17"
         }
     }
 
     withType<JavaCompile>().configureEach {
-        sourceCompatibility = JavaVersion.VERSION_11.name
-        targetCompatibility = JavaVersion.VERSION_11.name
+        sourceCompatibility = JavaVersion.VERSION_17.name
+        targetCompatibility = JavaVersion.VERSION_17.name
     }
 }
 

cache/build.gradle.kts

@@ -70,7 +70,7 @@ kotlin {
         }
     }
 
-    jvmToolchain(11)
+    jvmToolchain(17)
 }
 
 android {
@@ -92,8 +92,8 @@ android {
     }
 
     compileOptions {
-        sourceCompatibility = JavaVersion.VERSION_11
-        targetCompatibility = JavaVersion.VERSION_11
+        sourceCompatibility = JavaVersion.VERSION_17
+        targetCompatibility = JavaVersion.VERSION_17
     }
 }
 

cache/src/commonMain/kotlin/org/mobilenativefoundation/store/cache5/Cache.kt

@@ -14,7 +14,10 @@ interface Cache<Key : Any, Value : Any> {
      * @throws UncheckedExecutionException If an unchecked exception was thrown while loading the value.
      * @throws ExecutionError If an error was thrown while loading the value.
      */
-    fun getOrPut(key: Key, valueProducer: () -> Value): Value
+    fun getOrPut(
+        key: Key,
+        valueProducer: () -> Value,
+    ): Value
 
     /**
      * @return Map of the [Value] associated with each [Key] in [keys]. Returned map only contains entries already present in the cache.
@@ -26,7 +29,10 @@ interface Cache<Key : Any, Value : Any> {
      * If the cache previously contained a value associated with [key], the old value is replaced by [value].
      * Prefer [getOrPut] when using the conventional "If cached, then return. Otherwise create, cache, and then return" pattern.
      */
-    fun put(key: Key, value: Value)
+    fun put(
+        key: Key,
+        value: Value,
+    )
 
     /**
      * Copies all of the mappings from the specified map to the cache. The effect of this call is

cache/src/commonMain/kotlin/org/mobilenativefoundation/store/cache5/CacheBuilder.kt

@@ -19,43 +19,52 @@ class CacheBuilder<Key : Any, Output : Any> {
     internal var ticker: Ticker? = null
         private set
 
-    fun concurrencyLevel(producer: () -> Int): CacheBuilder<Key, Output> = apply {
-        concurrencyLevel = producer.invoke()
-    }
-
-    fun maximumSize(maximumSize: Long): CacheBuilder<Key, Output> = apply {
-        if (maximumSize < 0) {
-            throw IllegalArgumentException("Maximum size must be non-negative.")
+    fun concurrencyLevel(producer: () -> Int): CacheBuilder<Key, Output> =
+        apply {
+            concurrencyLevel = producer.invoke()
         }
-        this.maximumSize = maximumSize
-    }
 
-    fun expireAfterAccess(duration: Duration): CacheBuilder<Key, Output> = apply {
-        if (duration.isNegative()) {
-            throw IllegalArgumentException("Duration must be non-negative.")
+    fun maximumSize(maximumSize: Long): CacheBuilder<Key, Output> =
+        apply {
+            if (maximumSize < 0) {
+                throw IllegalArgumentException("Maximum size must be non-negative.")
+            }
+            this.maximumSize = maximumSize
         }
-        expireAfterAccess = duration
-    }
 
-    fun expireAfterWrite(duration: Duration): CacheBuilder<Key, Output> = apply {
-        if (duration.isNegative()) {
-            throw IllegalArgumentException("Duration must be non-negative.")
+    fun expireAfterAccess(duration: Duration): CacheBuilder<Key, Output> =
+        apply {
+            if (duration.isNegative()) {
+                throw IllegalArgumentException("Duration must be non-negative.")
+            }
+            expireAfterAccess = duration
         }
-        expireAfterWrite = duration
-    }
 
-    fun ticker(ticker: Ticker): CacheBuilder<Key, Output> = apply {
-        this.ticker = ticker
-    }
+    fun expireAfterWrite(duration: Duration): CacheBuilder<Key, Output> =
+        apply {
+            if (duration.isNegative()) {
+                throw IllegalArgumentException("Duration must be non-negative.")
+            }
+            expireAfterWrite = duration
+        }
 
-    fun weigher(maximumWeight: Long, weigher: Weigher<Key, Output>): CacheBuilder<Key, Output> = apply {
-        if (maximumWeight < 0) {
-            throw IllegalArgumentException("Maximum weight must be non-negative.")
+    fun ticker(ticker: Ticker): CacheBuilder<Key, Output> =
+        apply {
+            this.ticker = ticker
         }
 
-        this.maximumWeight = maximumWeight
-        this.weigher = weigher
-    }
+    fun weigher(
+        maximumWeight: Long,
+        weigher: Weigher<Key, Output>,
+    ): CacheBuilder<Key, Output> =
+        apply {
+            if (maximumWeight < 0) {
+                throw IllegalArgumentException("Maximum weight must be non-negative.")
+            }
+
+            this.maximumWeight = maximumWeight
+            this.weigher = weigher
+        }
 
     fun build(): Cache<Key, Output> {
         if (maximumSize != -1L && weigher != null) {