My first multiplatform library open for feedback. <https://kotlinlang.slack.com/archives/C0BJ0GTE2/p...

# multiplatform

[xxfast]

My first multiplatform library open for feedback.
https://kotlinlang.slack.com/archives/C0BJ0GTE2/p1662937749366859

[Michael Paus]

Shouldn’t

val storeOf: KStore<Pet> = store("path/to/file")

be

val store: KStore<Pet> = storeOf("path/to/file")

[xxfast]

Ah i messed up the docs - nice catch. It should be

val store: KStore<Pet> = store("path/to/file")

[Michael Paus]

According to https://github.com/xxfast/KStore#configurations it should be storeOf(…), shouldn’t it?

[xxfast]

it was before, but’ve changed the api to simply be just store()

[xxfast]

i might change my mind before this hits 1.0 😅

[xxfast]

wanted to be consistent with collection factory methods like listOf setOf etc. But this doesn’t really read well when it is just a path

val store: KStore<Pet> = storeOf(path)

[Michael Paus]

I would then change the example to

val petStore: KStore<Pet> = store("path/to/file")

in order to avoid any confusion. In practice you will mostly have several different stores anyway.

(I still like the storeOf variant though 😉)

[xxfast]

Done 👍 i’ve changed my mind and went with storeOf