search

ReactiveCircus / cache4k

In-memory Cache for Kotlin Multiplatform.
https://reactivecircus.github.io/cache4k/
Apache License 2.0
275 stars 20 forks source link

Add KeyedSynchronizer for synchronized cache loader #15

Closed dcvz closed 2 years ago

dcvz commented 2 years ago

Adds KeyedSynchronizer and tests to synchronize cache loader -- based on previous work by @ychescale9 .

dcvz commented 2 years ago

@ychescale9 looks like compilation was failing on JS due to test names. I've updated them now.

dcvz commented 2 years ago

Should also solve the concern in #14

dcvz commented 2 years ago

@ychescale9 Looks like detekt was failing, should be all green now 🤞

ychescale9 commented 2 years ago

Thanks! Will take a look on the weekend.

dcvz commented 2 years ago

Thanks! Realized detekt was still catching something, checked it locally to verify now.

ychescale9 commented 2 years ago

Thanks for contributing!