[whimsy] branch master updated: Does not make sense to keep total

[sebb]

This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git


The following commit(s) were added to refs/heads/master by this push:
     new 30499726 Does not make sense to keep total
30499726 is described below

commit 304997260b20969855361b032037c15a3dd19ebc
Author: Sebb <s...@apache.org>
AuthorDate: Tue May 3 17:13:07 2022 +0100

    Does not make sense to keep total
---
 tools/merge_subscriptions.rb | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/tools/merge_subscriptions.rb b/tools/merge_subscriptions.rb
index 20f85b71..2d1a1a11 100644
--- a/tools/merge_subscriptions.rb
+++ b/tools/merge_subscriptions.rb
@@ -49,11 +49,10 @@ def merge_files(old_host, new_host, out)
   end
 
   File.open(File.join(out, 'list-counts'), 'w') do |f|
-    total = counts.delete('total') # does not sort correctly
+    counts.delete('total') # does not sort correctly; and cannot be merged
     counts.sort.each do |k, v|
       f.puts "#{v} #{k}"
     end
-    f.puts("#{total} total") # add total back in
   end
 
   # merge list-flags