-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
showPlayersPosition(players) | ||
} | ||
|
||
override fun onCancelled(error: DatabaseError) { |
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.
Similar blocks of code found in 3 locations. Consider refactoring.
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 good! Great job!
Code Climate has analyzed commit a0eabe9 and detected 1 issue on this pull request. Here's the issue category breakdown:
The test coverage on the diff in this pull request is 94.2% (80% is the threshold). This pull request will bring the total coverage in the repository to 87.5% (0.0% change). View more on Code Climate. |
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.
LGTM, good job :)
replace hard-coded data in the scoreboard with our real database.
p.s, we still need to discuss how to calculate the ranking.