Rechee Jozil
05/29/2020, 8:50 PMzak.taccardi
05/29/2020, 8:51 PMhaving some kind of static global scopeNever do this. You make testing harder. Inject your scopes everywhere. Solves all kinds of issues
zak.taccardi
05/29/2020, 8:54 PMCoroutineScope(dispatchers.default)
. And we also inject our dispatchers too.
We disallow any reference to Dispatchers.XXX
outside of this file, as well as GlobalScope
Sinan Kozak
05/29/2020, 8:56 PMRechee Jozil
05/29/2020, 8:57 PMSinan Kozak
05/29/2020, 8:57 PMzak.taccardi
05/29/2020, 8:57 PMGlobalScope
? We honor system it because we’re small enough, but a lint warning would suffice if we were a larger teamRechee Jozil
05/29/2020, 8:58 PMRechee Jozil
05/29/2020, 9:13 PMzak.taccardi
05/29/2020, 9:13 PMzak.taccardi
05/29/2020, 9:14 PMSinan Kozak
05/30/2020, 7:08 AM