Turbolinks config requires turbolinks.js be precompiled

[ryanb]

Enabling the turbolinks config causes this error:

Sprockets::Rails::Helper::AssetNotPrecompiled - Asset was not declared to be precompiled in production.
Add `Rails.application.config.assets.precompile += %w( turbolinks.js )` to `config/initializers/assets.rb` and restart your server

This is because turbolinks.js is included as a separate script in the HTML when this config is enabled.

I thinks it's better for turbolinks to be included in application.js so it's managed by sprockets and not a separately loaded JS file. Then you could remove this turbolinks config and add documentation for enabling it by overriding application.js

[tian-im]

Yeah, that's one way to do it. And I'm thinking to use a different way, for example:

<%= javascript_include_tag 'turbolinks', 'data-turbolinks-track' => true, 'data-turbolinks-eval' => false if Rails.application.config.assets.precompile.include?('turbolinks.js') %>

Then once turbolinks is added to precompile assets, Wallaby will include it as well.

[tian-im]

But yeah, maybe leave it to the user is the right way to go.