Merge pull request #2724 from MikeMcQuaid/capital-desc

formula_desc_cop: desc should be capitalised.
This commit is contained in:
Mike McQuaid 2017-06-03 09:57:26 +01:00 committed by GitHub
commit b7d007fb50

View File

@ -32,13 +32,18 @@ module RuboCop
# Check if command-line is wrongly used in formula's desc
if match = regex_match_group(desc, /(command ?line)/i)
problem "Description should use \"command-line\" instead of \"#{match}\""
c = match.to_s.chars.first
problem "Description should use \"#{c}ommand-line\" instead of \"#{match}\""
end
if match = regex_match_group(desc, /^(an?)\s/i)
problem "Description shouldn't start with an indefinite article (#{match})"
end
if regex_match_group(desc, /^[a-z]/)
problem "Description should start with a capital letter"
end
# Check if formula's name is used in formula's desc
problem "Description shouldn't include the formula name" if regex_match_group(desc, /^#{@formula_name}\b/i)
end