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

1 files changed, 2 insertions, 10 deletions

diff --git a/Library/Homebrew/cmd/readall.rb b/Library/Homebrew/cmd/readall.rb
index 3591e0c09..0fa1f7c07 100644
--- a/Library/Homebrew/cmd/readall.rb
+++ b/Library/Homebrew/cmd/readall.rb
@@ -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