Closed antonis closed 2 weeks ago
2 Warnings | |
---|---|
:warning: | This PR is larger than 300 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews. |
:warning: | Class BlockType is missing tests, but unit-tests-exemption label was set to ignore this. |
Generated by :no_entry_sign: Danger
App Name | Jetpack | |
Flavor | Jalapeno | |
Build Type | Debug | |
Version | pr20711-9fd9d9a | |
Commit | 9fd9d9ad0cd5501c94653ac694e049817a71df1d | |
Direct Download | jetpack-prototype-build-pr20711-9fd9d9a.apk |
App Name | WordPress | |
Flavor | Jalapeno | |
Build Type | Debug | |
Version | pr20711-9fd9d9a | |
Commit | 9fd9d9ad0cd5501c94653ac694e049817a71df1d | |
Direct Download | wordpress-prototype-build-pr20711-9fd9d9a.apk |
Thank you for reviewing @jarvislin π
It seems that our CI/CD is unstable, I gonna re-run the task, please feel free to merge it after the build is done.
I'll run the tests again and if they fail I'll check if any change was introduced with the code. It's weird that https://github.com/wordpress-mobile/WordPress-Android/pull/20713 passed the CI checks with the 1st attempt π€
I'll run the tests again and if they fail I'll check if any change was introduced with the code.
This should be fixed with 9fd9d9ad0cd5501c94653ac694e049817a71df1d
thank you for the tip @jarvislin π
Issues
1 New issue
0 Accepted issues
Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code
Attention: Patch coverage is 2.40385%
with 203 lines
in your changes are missing coverage. Please review.
Project coverage is 40.65%. Comparing base (
575b40c
) to head (9fd9d9a
). Report is 2 commits behind head on feature/notifications_refresh_p2.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
As an initial step towards the Milestone screen revamp this PR refactors some of the related code to Kotlin.
To Test:
Regression Notes
Potential unintended areas of impact
What I did to test those areas of impact (or what existing automated tests I relied on)
What automated tests I added (or what prevented me from doing so)
PR Submission Checklist:
RELEASE-NOTES.txt
if necessary.Testing Checklist (strike-out the not-applying and unnecessary ones):