Add yarn install into bin/setup when not using importmap

After af7428c4ac the yarn install
instructions was dropped from bin/setup. This commmit adds it into
the setup script again if the user is not using importmap.
This commit is contained in:
Cadu Ribeiro 2022-01-18 00:38:58 -03:00
parent b589d519a6
commit 155b7570fa
2 changed files with 12 additions and 0 deletions

View File

@ -1,3 +1,9 @@
* Add JavaScript dependencies installation on bin/setup
Add `yarn install` to bin/setup when using esbuild, webpack, or rollout.
*Carlos Ribeiro*
* Use `controller_class_path` in `Rails::Generators::NamedBase#route_url`
The `route_url` method now returns the correct path when generating

View File

@ -15,6 +15,12 @@ FileUtils.chdir APP_ROOT do
puts "== Installing dependencies =="
system! "gem install bundler --conservative"
system("bundle check") || system!("bundle install")
<% if ["webpack", "esbuild", "rollup"].include?(options.javascript) -%>
# Install JavaScript dependencies
system("yarn check --check-files") || system!("yarn install")
<% end -%>
<% unless options.skip_active_record? -%>
# puts "\n== Copying sample files =="