From fe2bcd352899cd402091c2815846d17a0eb1369b Mon Sep 17 00:00:00 2001 From: Takashi Kokubun Date: Thu, 22 Dec 2022 16:33:44 -0800 Subject: Check `added` for the second sub! properly Even if the first sub! modifies `news`, when `added` is empty, it always ended up skipping `File.write("NEWS.md", news)` because of the `next`. This commit fixes the problem. --- .github/workflows/bundled_gems.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to '.github/workflows/bundled_gems.yml') diff --git a/.github/workflows/bundled_gems.yml b/.github/workflows/bundled_gems.yml index c273c9d57d..6de4e02de7 100644 --- a/.github/workflows/bundled_gems.yml +++ b/.github/workflows/bundled_gems.yml @@ -81,7 +81,7 @@ jobs: news.sub!(/^\*( +)The following default gems are now bundled gems\.\n+\K(?: \1\*( +).*\n)*/) do mark = "#{$1} *#{$2}" added.map {|g, v|"#{mark}#{g} #{v}\n"}.join("") - end or next if added + end or next unless added.empty? File.write("NEWS.md", news) end shell: ruby {0} -- cgit v1.2.3