Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Output thousands and millions of hits as integers #190

Merged
merged 2 commits into from
Apr 14, 2016
Merged

Output thousands and millions of hits as integers #190

merged 2 commits into from
Apr 14, 2016

Conversation

chillpop
Copy link
Contributor

Previously (in #179), hit counts in the thousands or millions were output as floats and lower hit counts were output as integers. This change makes all hit counts consistent.

@chillpop
Output thousands and millions of hits as integers
c95f3d1 
Previously, hit counts in the thousands or millions were output as floats and lower hit counts were output as integers.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 95.057% when pulling c95f3d1 on chillpop:hit-count-as-integer into 885bb4a on SlatherOrg:master.

@neonichu
Copy link
Member

LGTM 👍 — could you add an entry to the changelog, please?

@chillpop
Copy link
Contributor Author

Thanks, changelog updated.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 95.057% when pulling dc42ae4 on chillpop:hit-count-as-integer into 885bb4a on SlatherOrg:master.

@neonichu neonichu merged commit 0db7fdf into SlatherOrg:master Apr 14, 2016
@chillpop chillpop deleted the hit-count-as-integer branch April 14, 2016 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@chillpop @coveralls @neonichu