Closed MohitMaliFtechiz closed 1 month ago
Attention: Patch coverage is 64.03941%
with 73 lines
in your changes missing coverage. Please review.
Please upload report for BASE (
main@453c1d3
). Learn more about missing BASE report. Report is 45 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@kelson42 This PR is now ready for review.
Fixes #3478
The
BuildConfig
file is by default off in the latest gradle version, so we have enabled this in ourAllProjectConfigure
class to allow this class for both modules.Upgraded the
kotlin
version to1.9.20
.Upgraded the
dagger
version to2.48.1
.Upgraded the
gradle
version to8.1.3
.Upgraded the
webkit
version to1.11.0
.Upgraded the
jacoco
version to0.8.12
.Upgraded the
uiautomator
version to2.3.0
.Upgraded the
appcompat
version to1.7.0
.Upgraded the coroutines version to
1.8.1
.Upgraded the
androidx.test:runner
version to1.6.1
.Upgraded the
androidx.test:core
version to1.6.1
.Upgraded the
androidx.test:orchestrator
version to1.5.0
.Upgraded the
org.junit.jupiter:junit-jupiter
version to5.11.0
.Upgraded the
org.assertj:assertj-core
version to3.26.3
.Upgraded the
androidx.annotation:annotation
version to1.6.0
.Improved the SearchStateTest.
Fixed some deprecated code which is deprecated in new dependencies.
Removed the
butterknife
dependency from the project since it is not compatible with the new gradle so we have used the binding for getting views instead of this deps.Refactored our code to support Android 14 properly.
Added Android 14 emualtor in CI to run the test cases on Android 14.
Fixed memory leaks in the application that come after upgrading the gradle.
Updated the
README
file to show that now our project requires theJAVA 17
instead ofJAVA 11
.Updated the pre-commit failed message to show the error information properly