FIX: Don't install test gems in production.

FIX: Don’t install test gems in production.

This fixes an incorrect usage of the --without option. As per the documentation, it takes groups seperated by a space --without=GROUP[ GROUP...]. Specifying the option twice meant we were overriding the first which lead the this bug.

diff --git a/lib/docker_manager/upgrader.rb b/lib/docker_manager/upgrader.rb
index 194c13b..26ee44d 100644
--- a/lib/docker_manager/upgrader.rb
+++ b/lib/docker_manager/upgrader.rb
@@ -70,7 +70,7 @@ class DockerManager::Upgrader
       percent(20 * (index + 1) / @repos.size)
     end
 
-    run("bundle install --deployment --without test --without development")
+    run("bundle install --deployment --jobs 4 --without test development")
     percent(30)
     run("SKIP_POST_DEPLOYMENT_MIGRATIONS=1 bundle exec rake multisite:migrate")
     percent(40)

GitHub sha: 79aa85a8

2 Likes