Skip to content

Commit 4519490

Browse files
committed
Don't pre-create asset manifests
This causes a file generation conflict in Rails 7.2, as they seem to already exist. We don't want to overwrite whatever there is. ``` conflict app/assets/config/manifest.js Overwrite /Users/pirj/source/rspec-dev/repos/rspec-rails/tmp/example_app/app/assets/config/mani Overwrite /Users/pirj/source/rspec-dev/repos/rspec-rails/tmp/example_app/app/assets/config/manifest.js? (enter "h" for help) [Ynaqdhm] y force app/assets/config/manifest.js conflict app/assets/stylesheets/application.css Overwrite /Users/pirj/source/rspec-dev/repos/rspec-rails/tmp/example_app/app/assets/stylesheets Overwrite /Users/pirj/source/rspec-dev/repos/rspec-rails/tmp/example_app/app/assets/stylesheets/application.css? (enter "h" for help) [Ynaqdhm] y force app/assets/stylesheets/application.css ```
1 parent 2d6d0ac commit 4519490

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

example_app_generator/generate_app.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@
6767

6868
copy_file capybara_backport_path, 'spec/support/capybara.rb'
6969

70-
if Rails::VERSION::STRING > '7'
70+
if Rails::VERSION::STRING > '7' && Rails::VERSION::STRING < '7.2'
7171
create_file 'app/assets/config/manifest.js' do
7272
"//= link application.css"
7373
end

0 commit comments

Comments
 (0)