@pablobm commented on this pull request.


> @@ -16,6 +16,7 @@ FileUtils.chdir APP_ROOT do
 
   puts "== Installing dependencies =="
   system("bundle check") || system!("bundle install")
+  system("bin/yarn check") || system!("bin/yarn install")

Good call. I suspect that it's a bit faster as it won't traverse the dependency 
graph twice. In any case, when in doubt I prefer to do exactly as provided by 
Rails. Changed now.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6432#discussion_r2413274502
You are receiving this because you are subscribed to this thread.

Message ID: 
<openstreetmap/openstreetmap-website/pull/6432/review/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to