Skip to content

Commit 44d367e

Browse files
committed
Deprecate #plural.
1 parent 00306a4 commit 44d367e

File tree

4 files changed

+8
-16
lines changed

4 files changed

+8
-16
lines changed

Library/Homebrew/compat/utils.rb

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,3 +21,8 @@ def puts_columns(items)
2121
odeprecated "puts_columns", "puts Formatter.columns"
2222
puts Formatter.columns(items)
2323
end
24+
25+
def plural(n, s = "s")
26+
odeprecated "#plural", "Formatter.pluralize"
27+
n == 1 ? "" : s
28+
end

Library/Homebrew/test/utils_spec.rb

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -219,13 +219,6 @@ def esc(code)
219219
end
220220
end
221221

222-
specify "#plural" do
223-
expect(plural(1)).to eq("")
224-
expect(plural(0)).to eq("s")
225-
expect(plural(42)).to eq("s")
226-
expect(plural(42, "")).to eq("")
227-
end
228-
229222
specify "#disk_usage_readable" do
230223
expect(disk_usage_readable(1)).to eq("1B")
231224
expect(disk_usage_readable(1000)).to eq("1000B")

Library/Homebrew/utils.rb

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -139,10 +139,6 @@ def pretty_duration(s)
139139
res << Formatter.pluralize(s, "second")
140140
end
141141

142-
def plural(n, s = "s")
143-
n == 1 ? "" : s
144-
end
145-
146142
def interactive_shell(f = nil)
147143
unless f.nil?
148144
ENV["HOMEBREW_DEBUG_PREFIX"] = f.prefix

Library/Homebrew/utils/formatter.rb

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -96,11 +96,9 @@ def pluralize(count, singular, plural = nil)
9696

9797
*adjectives, noun = singular.split(" ")
9898

99-
if plural.nil?
100-
plural = {
101-
"formula" => "formulae",
102-
}.fetch(noun, "#{noun}s")
103-
end
99+
plural ||= {
100+
"formula" => "formulae",
101+
}.fetch(noun, "#{noun}s")
104102

105103
words = adjectives << plural
106104

0 commit comments

Comments
 (0)