In 'readall.rb', replaced multi-step 'each' loop with one-line method chain of Ruby enumerator methods

This commit is contained in:
Richie Thomas 2017-10-11 10:51:19 -04:00 committed by richiethomas
parent 56458f03fc
commit 9ef82ae1e2

View File

@ -13,16 +13,8 @@ module Homebrew
def readall
if ARGV.include?("--syntax")
ruby_files = []
scan_files = %W[
#{HOMEBREW_LIBRARY}/*.rb
#{HOMEBREW_LIBRARY}/Homebrew/**/*.rb
]
Dir.glob(scan_files).each do |rb|
next if rb.include?("/vendor/")
next if rb.include?("/cask/")
ruby_files << rb
end
scan_files = "#{HOMEBREW_LIBRARY}/Homebrew/**/*.rb"
ruby_files = Dir.glob(scan_files).reject{|file| file =~ /vendor|cask/ }
Homebrew.failed = true unless Readall.valid_ruby_syntax?(ruby_files)
end