Skip to content
This repository has been archived by the owner on May 9, 2022. It is now read-only.

Adds null checks #23

Merged
merged 1 commit into from
Feb 28, 2018
Merged

Adds null checks #23

merged 1 commit into from
Feb 28, 2018

Conversation

NejcZdovc
Copy link
Contributor

  • Adds null check in topN function
  • Fixes one typo found in the code vists -> visits

@NejcZdovc NejcZdovc self-assigned this Feb 28, 2018
@NejcZdovc NejcZdovc requested a review from mrose17 February 28, 2018 06:34
@NejcZdovc NejcZdovc mentioned this pull request Feb 28, 2018
Closed
mrose17
mrose17 reviewed Feb 28, 2018
View reviewed changes
Copy link
Contributor

@mrose17 mrose17 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@mrose17 mrose17 merged commit 13686d8 into brave-intl:master Feb 28, 2018
NejcZdovc added a commit that referenced this pull request Mar 19, 2018
@NejcZdovc
Adds null checks (#23)
80b9aa9
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mrose17 mrose17 mrose17 approved these changes

Assignees

@NejcZdovc NejcZdovc

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@NejcZdovc @mrose17