The change to the client store initialization looks good to me as it is – the changes in the migrations aren't quite right, what we are actually working with there is a sqlite3.Connection object, so I will fix those type hints. Anyway, good catch, and, I am actually considering using peewee or something to support other database backends, at which point this might in fact become a peewee.Database object.
Again, thanks for your contribution – I will merge this now, then fix the type hints myself.
Hi, thanks for those fixes!
The change to the client store initialization looks good to me as it is – the changes in the migrations aren't quite right, what we are actually working with there is a sqlite3.Connection object, so I will fix those type hints. Anyway, good catch, and, I am actually considering using peewee or something to support other database backends, at which point this might in fact become a peewee.Database object.
Again, thanks for your contribution – I will merge this now, then fix the type hints myself.