admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

Merge pull request #1311 from MikeMcQuaid/audit-no-trailing-whitespace

Browse Source 
audit: don't complain about trailing whitespace.
This commit is contained in:
Mike McQuaid 2016-10-18 08:38:39 +01:00 committed by GitHub
commit 2da012d84d
1 changed files with 1 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
Library/Homebrew/dev-cmd/audit.rb
View File

@ -736,7 +736,7 @@ class FormulaAuditor
problem "require \"language/go\" is unnecessary unless using `go_resource`s" problem "require \"language/go\" is unnecessary unless using `go_resource`s"
end end
def audit_line(line, lineno) def audit_line(line, _lineno)
if line =~ /<(Formula|AmazonWebServicesFormula|ScriptFileFormula|GithubGistFormula)/ if line =~ /<(Formula|AmazonWebServicesFormula|ScriptFileFormula|GithubGistFormula)/
problem "Use a space in class inheritance: class Foo < #{$1}" problem "Use a space in class inheritance: class Foo < #{$1}"
end end
@ -817,9 +817,6 @@ class FormulaAuditor
# Commented-out depends_on # Commented-out depends_on
problem "Commented-out dep #{$1}" if line =~ /#\s*depends_on\s+(.+)\s*$/ problem "Commented-out dep #{$1}" if line =~ /#\s*depends_on\s+(.+)\s*$/
# No trailing whitespace, please
problem "#{lineno}: Trailing whitespace was found" if line =~ /[\t ]+$/
if line =~ /if\s+ARGV\.include\?\s+'--(HEAD|devel)'/ if line =~ /if\s+ARGV\.include\?\s+'--(HEAD|devel)'/
problem "Use \"if build.#{$1.downcase}?\" instead" problem "Use \"if build.#{$1.downcase}?\" instead"
end end