Closed vmadalin closed 3 years ago
Merging #1 (34ad7bb) into master (3a3d595) will increase coverage by
54.18%
. The diff coverage is54.18%
.
@@ Coverage Diff @@
## master #1 +/- ##
===========================================
+ Coverage 0.00% 54.18% +54.18%
===========================================
Files 14 9 -5
Lines 486 227 -259
Branches 58 46 -12
===========================================
+ Hits 0 123 +123
+ Misses 486 81 -405
- Partials 0 23 +23
Impacted Files | Coverage Δ | |
---|---|---|
...vmadalin/easypermissions/dialogs/SettingsDialog.kt | 0.00% <0.00%> (ø) |
|
...vmadalin/easypermissions/utils/AnnotationsUtils.kt | 41.66% <41.66%> (ø) |
|
...adalin/easypermissions/models/PermissionRequest.kt | 48.88% <48.88%> (ø) |
|
...madalin/easypermissions/dialogs/RationaleDialog.kt | 53.12% <53.12%> (ø) |
|
...sypermissions/helpers/FragmentPermissionsHelper.kt | 85.71% <85.71%> (ø) |
|
...va/com/vmadalin/easypermissions/EasyPermissions.kt | 89.79% <89.79%> (ø) |
|
.../easypermissions/helpers/base/PermissionsHelper.kt | 92.30% <92.30%> (ø) |
|
...sypermissions/helpers/ActivityPermissionsHelper.kt | 100.00% <100.00%> (ø) |
|
...ions/helpers/AppCompatActivityPermissionsHelper.kt | 100.00% <100.00%> (ø) |
|
... and 8 more |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 3a3d595...34ad7bb. Read the comment docs.
Original PR
https://github.com/googlesamples/easypermissions/pull/289
Description
Migrate all entire project to
Kotlin
1.3.50, but keeping the integration concept. These are some of the things that have been done:ktlint
,detekt
,spotless
for static analysis code adding them toTravis
Rationale/Settings
dialogs displaying anAlertDialog
LowAPiPermissions
for avoid throw exception to integrator on their codeOverview
Architecture:
Coverage: