#comment Generic `test` which is appropriate for plain Java, has been replaced with `testDebugUnitTest` which executes test of only one configuration, instead of for every configuration
Affected: CI tests
#comment A custom Detekt configuration has been built based on team evaluations.
detekt.yml has been moved from /app/config/detekt/detekt.yml to /detekt/config.yml, so Detekt plugin settings need to be updated accordingly.
Detekt has been updated to 1.8.0 for support more rules.
MaxIssues has been raised to 10660 for match the new configuration
Affected: Detekt plugin settings
Ticket: MAILAND-590
#comment Added <style> html tag to our blacklist of html tags, so we always strip it if the message contains it.
Affected: Maybe some newsletters won't render the same way as before
MAILAND-587
#comment The sentry config file wasn't inside the root folder (it was deeper in the hierarchy) which was leading to a build failure for release builds
Affected: Sentry configuration
Ticket: none
#comment Some attributes have been fixed on messages_list_item_new.xml since they had a wrong format which was leading to a build failure for Release buils
Affected: messages_list_item_new.xml
Ticket: none
Now `develop` is aligned with `prerelease` and other branches, it contains the configuration for build release versions of the app.
Also many BuildConfig fields have been removed: some of them have been replaced by `BuildConfig.DEBUG`, instead of make that differentiation on branches; other didn't have sense to exist, so a comment has been left there.
Sentry logs ( before not present on `develop` ) have been added with a check `if (!BuildConfig.DEBUG)` for bein' executed only on release version.
Gradle version has been updated to 6.3 for being able to work with JDK 14.
Affected: Gradle version, release config for `develop`, Sentry logs on `develop`, BuildConfig fields
Ticket: none