Merge pull request #4891 from zgracem/add-commas-to-analytics
use 'number_readable' when displaying analytics
This commit is contained in:
commit
1b45666518
@ -207,7 +207,7 @@ module Homebrew
|
|||||||
value.each do |range, results|
|
value.each do |range, results|
|
||||||
oh1 "#{category} (#{range})"
|
oh1 "#{category} (#{range})"
|
||||||
results.each do |name_with_options, count|
|
results.each do |name_with_options, count|
|
||||||
puts "#{name_with_options}: #{count}"
|
puts "#{name_with_options}: #{number_readable(count)}"
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
@ -216,7 +216,7 @@ module Homebrew
|
|||||||
|
|
||||||
json["analytics"].each do |category, value|
|
json["analytics"].each do |category, value|
|
||||||
analytics = value.map do |range, results|
|
analytics = value.map do |range, results|
|
||||||
"#{results.values.inject("+")} (#{range})"
|
"#{number_readable(results.values.inject("+"))} (#{range})"
|
||||||
end
|
end
|
||||||
puts "#{category}: #{analytics.join(", ")}"
|
puts "#{category}: #{analytics.join(", ")}"
|
||||||
end
|
end
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user