home-assistant / home-assistant/android
Review coroutine scopes in SensorManager
- Dominant language
- Kotlin
- Stars
- 3.9k
- Forks
- 1.1k
- Avg merge
- 1d 12h
- Merged PRs (30d)
- 81
Description
On the effort of making sure that DB calls are not made on the main thread on this PR https://github.com/home-assistant/android/pull/5517 we've started discussing about the general scope usage in it. It exposes a Main scope and everything is launch into it, we should double check if it still make sense and if we can't put some work into an IO scope.
Alongside with this we need to double check the usage of `Job` vs `SupervisedJob`.
For more context: https://github.com/home-assistant/android/pull/5517#discussion_r2223406457
Any attempt to work on this should result into cleaning the TODO in the code base associated with this issue.
Contributor guide
Assessment
This issue has not been assessed yet.