Add commit attribute to AndroidSettings.Factory

AndroidSettings constructor allows to set a commit flag.

I am missing the same configuration option when using AndroidSettings.Factory.

As the fun create(String?) is defined in the parent Settings.Factory interface, the commit option would need to be set in AndroidSettings.Factory constructor.

If we can agree on this I can to the PR.

[russhwolf]

Platform-specific settings should be configured in platform code, not common code. Otherwise you end up with a bunch of extra flags in common code that don’t do anything most of the time, and that makes for a confusing API.

The purpose of no-arg is to provide a reasonable default configuration where you don’t need to do anything extra. If those default settings don’t fit your needs, then you shouldn’t be using no-arg.

[russhwolf]

There’s a bit of a combinatorial explosion when trying to support every option for both constructors and factories. I haven’t been very focused on the Factory implementations since you can fairly easily create your own if you need different options. I’ve also become less and less convinced over time that having a Factory interface is really that useful compared to just supplying your own () -> Settings or (String) -> Settings lambda as needed.

At some point I want to do some deeper rethinking of how initialization works, and have been thinking about doing a DSL builder style like Ktor and kotlinx.serialization use. See #65.