Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Skip to content

4339-Set_log_level_for_release_build

Fahim Salam Chowdhuryrequested to merge
4339-Set_log_level_for_release_build into main

Description

co.trikita.Log provides set log level support, which android.util.Log is missing. Again, co.trikita.Log maintains the same interface structure as android.util.Log, so only updating the import statement is only needed to update the code structure. Setting up log level to only error for release builds will stop printing sophisticated info into the logcat.

Screenshots

Technical details

  • Didn't use Timber or any other log libraries, because, those don't follow the same android.util.Log structure. So using any one of them will require also code statement changes that may cause merge conflict on updates with upstream tasks.

  • Ignore the pipeline failed issue

Tests

Issues

10 commandments of code reviews

👪 ❤ https://gitlab.e.foundation/internal/wiki/-/wikis/development/code-review

Edited by Fahim Salam Chowdhury

Merge request reports