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

Use relevant methods to consolidate logic in text_cop.rb

Browse Source
This commit is contained in:
Gautham Goli 2017-05-30 01:24:17 +05:30
parent 88f21b84f4
commit cfbdc17cb7
1 changed files with 20 additions and 38 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

58
Library/Homebrew/rubocops/text_cop.rb
View File

@ -10,61 +10,43 @@ module RuboCop
problem "Please set plist_options when using a formula-defined plist." problem "Please set plist_options when using a formula-defined plist."
end end
if depends_on?(body_node, "openssl") && depends_on?(body_node, "libressl") if depends_on?("openssl") && depends_on?("libressl")
problem "Formulae should not depend on both OpenSSL and LibreSSL (even optionally)." problem "Formulae should not depend on both OpenSSL and LibreSSL (even optionally)."
end end
if method_called_ever?(body_node, :virtualenv_create) || if method_called_ever?(body_node, :virtualenv_create) ||
method_called_ever?(body_node, :virtualenv_install_with_resources) method_called_ever?(body_node, :virtualenv_install_with_resources)
resource_calls = find_every_method_call_by_name(body_node, :resource) find_method_with_args(body_node, :resource, "setuptools") do
resource_calls.each do |m| problem "Formulae using virtualenvs do not need a `setuptools` resource."
if parameters_passed?(m, "setuptools")
problem "Formulae using virtualenvs do not need a `setuptools` resource."
end
end end
end end
unless method_called_ever?(body_node, :go_resource) unless method_called_ever?(body_node, :go_resource)
# processed_source.ast is passed instead of body_node because `require` would be outside body_node # processed_source.ast is passed instead of body_node because `require` would be outside body_node
require_calls = find_every_method_call_by_name(processed_source.ast, :require) find_method_with_args(processed_source.ast, :require, "language/go") do
require_calls.each do |m| problem "require \"language/go\" is unnecessary unless using `go_resource`s"
if parameters_passed?(m, "language/go")
problem "require \"language/go\" is unnecessary unless using `go_resource`s"
end
end end
end end
factory_calls = find_every_method_call_by_name(body_node, :factory) find_instance_method_call(body_node, "Formula", :factory) do
unless factory_calls.nil? problem "\"Formula.factory(name)\" is deprecated in favor of \"Formula[name]\""
factory_calls.each do |m|
if !m.children.empty? && m.children[0] && string_content(m.children[0]) == "Formula"
offending_node(m)
problem "\"Formula.factory(name)\" is deprecated in favor of \"Formula[name]\""
end
end
end end
xcodebuild_calls = find_every_method_call_by_name(body_node, :xcodebuild) find_every_method_call_by_name(body_node, :xcodebuild).each do |m|
unless xcodebuild_calls.nil? next if parameters_passed?(m, /SYMROOT=/)
xcodebuild_calls.each do |m| problem 'xcodebuild should be passed an explicit "SYMROOT"'
params = parameters(m).map { |param| string_content(param) }
if params.none? { |param| param.include?("SYMROOT=") } || params.empty?
offending_node(m)
problem 'xcodebuild should be passed an explicit "SYMROOT"'
end
end
end end
system_calls = find_every_method_call_by_name(body_node, :system) find_method_with_args(body_node, :system, "xcodebuild") do
return if system_calls.nil? problem %q(use "xcodebuild *args" instead of "system 'xcodebuild', *args")
system_calls.each do |m| end
if parameters_passed?(m, "go", "get")
problem "Formulae should not use `go get`. If non-vendored resources are required use `go_resource`s." find_method_with_args(body_node, :system, "scons") do
elsif parameters_passed?(m, "xcodebuild") problem "use \"scons *args\" instead of \"system 'scons', *args\""
problem %q(use "xcodebuild *args" instead of "system 'xcodebuild', *args") end
elsif parameters_passed?(m, "scons")
problem "use \"scons *args\" instead of \"system 'scons', *args\"" find_method_with_args(body_node, :system, "go", "get") do
end problem "Formulae should not use `go get`. If non-vendored resources are required use `go_resource`s."
end end
end end
end end