all application.(js,css) files pulled in

[scashin133]

I have a bower depedency on jquery-sortable, which includes application.(css.sass,js) files.

The default precompile logic

sprockets-rails/lib/sprockets/railtie.rb

Line 69 in 27c6441

config.assets.precompile = [LOOSE_APP_ASSETS, /(?:\/|\\|\A)application\.(css|js)$/]

causes all application css and js files to be precompiled which was not the desired effect for this library.

2.2.0 :001 > 'vendor/assets/bower_components/jquery-sortable/source/js/application.js' =~ /(?:\/|\\|\A)application\.(css|js)$/
 => 56

Was this intentional?

There was an issue filed with jquery-sortable johnny/jquery-sortable#134 asking them to remove it but I didn't understand why it was precompiling it in the first place.

[josh]

I definitely agree with you and I attempted to "fix" it recently, but apparently people are depending on this sort of thing. It ended up reverting the patch in the 2.x release. But I'm planning to have it locked down in the next major release 3.x which can have backwards incompatible changes. Thanks!

[scashin133]

Sounds good just wanted to make sure.

On Monday, January 26, 2015, Joshua Peek [email protected] wrote:

I definitely agree with you and I attempted to "fix" it recently, but
apparently people are depending on this sort of thing. But I'm planning to
have it locked down in the next major release 3.x which can have backwards
incompatible changes. Thanks!

—
Reply to this email directly or view it on GitHub
#218 (comment)
.

http://about.me/seancashin/bio

[bborn]

+1 also think I'm running into this. bower install jquery-sortable brings in all the development files (sass, etc.) which then don't precompile correctly. Wouldn't it be better to only include the compiled js and css files?