Igor Wojda
Igor Wojda
eg. `fun onTextChanged(listener: (text: CharSequence?, start: Int, before: Int, count: Int) -> Unit) { }` this should allow to better code completion in IDE
Comply with AS warning `viewLifecycleOwner` must be used instead of `this`, because the app may crash in some cases 
Google recommends enabling parallel JVM garbage collector to speed up builds // https://developer.android.com/studio/build/optimize-your-build?utm_source=android-studio#configure-gc
It would be nice to convert one of the module to on-demand-delivery so it will be downloaded when it is needed https://developer.android.com/guide/app-bundle/on-demand-delivery Additionally we could have custom Progress fragment https://developer.android.com/guide/navigation/navigation-dynamic#customize
Some elements within `android` block are duplicated in each module. Ideally this configuration should be applied only once (and work all the times, even for future modules). ``` android {...
`compileOptions` and `kotlinOptions` block are duplicated in each module. Ideally this configuration should be applied only once (and work all the times, even for future modules). ``` compileOptions { sourceCompatibility...
We could add some UI test to the project (Espresso?) eg. basic tests like - run the app - navigate to details of first item - navigate to favourites/profile
[ArchUnit](https://www.archunit.org/) is a framework that allows to test app architecture. It looks like it's not suited for android, so some research, experimentation and potentially contribution to `ArchUnit` may be required...
We could write custom [detekt](https://arturbosch.github.io/detekt/) rule. It can be any rule, but one example that comes to my head is rule that verifies clean architecture layers (verify `dependency rule` correctness)...