From 06cd167f2ba86020347d351057be98359e1ab6ce Mon Sep 17 00:00:00 2001 From: Matthew Abbott Date: Thu, 31 Mar 2016 21:15:22 -0500 Subject: [PATCH] Remote empty sections before joining them together Previously, a comma was being printed on the last line of every section, even if that section did not contribute any rows to the formatted output (e.g., no rows within that section had the tags needed to be output during this run). When such a zero-row section happens, it results in an line with only a comma on it, which is not valid json. This can make incremental adjustment of the jquery files rather tedious, even when using gui tools to quickly reject changes. --- lib/twine/formatters/jquery.rb | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/twine/formatters/jquery.rb b/lib/twine/formatters/jquery.rb index 8d60116..bfe8d4c 100644 --- a/lib/twine/formatters/jquery.rb +++ b/lib/twine/formatters/jquery.rb @@ -50,6 +50,7 @@ module Twine def format_sections(twine_file, lang) sections = twine_file.sections.map { |section| format_section(section, lang) } + sections.delete_if &:empty? sections.join(",\n\n") end