search

russhwolf / multiplatform-settings

A Kotlin Multiplatform library for saving simple key-value data
Apache License 2.0
1.69k stars 67 forks source link

Consider changing Level of ExperimentalListener to WARNING #27

Closed yshrsmz closed 5 years ago

yshrsmz commented 5 years ago

as Level.ERROR cannot be suppressed, and we ended up annotating every class with ExperimentalListener across all modules.

russhwolf commented 5 years ago

Yeah this is a good idea. I have the annotations there because I want to explicitly mark that API as less stable than everything else, but I also want to reduce barriers-to-entry in general because the Multiplatform ecosystem is already hard enough to get into. I'll reduce experimental annotation levels next time I do an update.

In the meantime, you can use @UseExperimental at the immediate use-site of an Experimental-annotated declaration to avoid leaking annotations up your entire call hierarchy.

yshrsmz commented 5 years ago

you can use [at]UseExperimental

That's great, thanks!

russhwolf commented 5 years ago

Level of all experimental annotations is reduced to Warning in 0.3.2

yshrsmz commented 5 years ago

Thanks!