-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks pretty good! In addition to my comments, please run the pre-push hook on this and address the linting errors.
You can also use ktlint -F
to fix most of them automatically.
app/src/androidTest/java/org/mozilla/fenix/ui/robots/TabsRobot.kt
Outdated
Show resolved
Hide resolved
app/src/androidTest/java/org/mozilla/fenix/ui/robots/SearchRobot.kt
Outdated
Show resolved
Hide resolved
0d1ea4e
to
5b4adb1
Compare
Codecov Report
@@ Coverage Diff @@
## master #4770 +/- ##
=========================================
Coverage 11.91% 11.91%
Complexity 245 245
=========================================
Files 247 247
Lines 10231 10231
Branches 1511 1511
=========================================
Hits 1219 1219
Misses 8932 8932
Partials 80 80 Continue to review full report at Codecov.
|
@sv-ohorvath once @sblatz comments are addressed and the tests pass on Firebase, let me know and I will merge 👍 |
5b4adb1
to
59d5eb4
Compare
@rpappalax I've run ktlint -F and the tests pass. This can be merged, thank you! |
ran the ktlint -F command automatically
Pull Request checklist