From fcf77c5ce6a54121c42fb8279ce88cad722724b0 Mon Sep 17 00:00:00 2001
From: Calen Pennington <calen.pennington@gmail.com>
Date: Wed, 17 Apr 2013 13:02:50 -0400
Subject: [PATCH] Run pylint in fewer larger batches

---
 rakefile | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/rakefile b/rakefile
index d6571118ca..6ed1ecce77 100644
--- a/rakefile
+++ b/rakefile
@@ -141,16 +141,20 @@ end
 
     desc "Run pylint on all #{system} code"
     task "pylint_#{system}" => report_dir do
-        Dir["#{system}/*.py", "#{system}/djangoapps/*", "#{system}/lib/*"].each do |app|
-            pythonpath_prefix = "PYTHONPATH=#{system}:#{system}/djangoapps:#{system}/lib:common/djangoapps:common/lib"
-            app = File.basename(app)
+        apps = Dir["#{system}/*.py", "#{system}/djangoapps/*", "#{system}/lib/*"].map do |app|
+            File.basename(app)
+        end.select do |app|
+            app !=~ /.pyc$/
+        end.map do |app|
             if app =~ /.py$/
-                app = app.gsub('.py', '')
-            elsif app =~ /.pyc$/
-                next
+                app.gsub('.py', '')
+            else
+                app
             end
-            sh("#{pythonpath_prefix} pylint --rcfile=.pylintrc -f parseable #{app} | tee #{report_dir}/#{app}.pylint.report")
         end
+
+        pythonpath_prefix = "PYTHONPATH=#{system}:#{system}/djangoapps:#{system}/lib:common/djangoapps:common/lib"
+        sh("#{pythonpath_prefix} pylint --rcfile=.pylintrc -f parseable #{apps.join(' ')} | tee #{report_dir}/pylint.report")
     end
     task :pylint => "pylint_#{system}"