Expose check_same_thread option for sqlite connection

[jamespeacock]

Set to False by default but now it is a configurable option

[coveralls]

Pull Request Test Coverage Report for Build 166

• 7 of 7 (100.0%) changed or added relevant lines in 3 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.04%) to 63.596%

---

Totals
Change from base Build 164:	0.04%
Covered Lines:	573
Relevant Lines:	901

---

💛 - Coveralls

[reece]

@jamespeacock Thanks for the PR!

What do you think about requiring that check_same_thread = True if writable == True? I'm assuming that thread safety only matters when the repo is writable (true?).

[jamespeacock]

I think that is super reasonable and will safeguard from improper use.
Does the hgvs package have any usages of SeqRepo with writeable=True?
I'm mostly thinking about implications for users of hgvs if this change by default turns the same thread check off.

Will update the PR with this change!

[reece]

Fabulous. Thanks again, James!
No, hgvs uses seqrepo in the default read-only mode.
In fact, the only time the repo is writable in any of my uses or by biocommons code is during loading.