Do not run yarn install on every assets:precompile

[Gargron]

At least add an option to disable that behaviour. Usually you have a separate step for making sure bundler/yarn dependencies are installed - I'd say in most cases. So this invocation is just wasting a few seconds in the best case, and in the worst case as I'm experiencing right now it's triggering post-install scripts every time.

[himdel]

IMHO dealing yarn should never even be webpacker's job. Don't tell me how to download my dependencies, just compile them when they are there. (Since most projects already had to deal with this and have their own solutions anyway.)

[gauravtiwari]

@Gargron @himdel That makes sense. I guess it should behave like bundler. This needs updating at two places - on Rails side and other on Webpacker - basically need to remove precompile hook enhancement.

[mipearson]

I tried working around this using the same workaround described at #440, but for some reason the yarn:install dep is added at some point after my own workaround is called.

Rake is confusing :(

[Gargron]

Afaik this is the line that does it: https://github.com/rails/webpacker/blob/master/lib/tasks/webpacker/compile.rake#L32 It also checks for something present in Rails 5.1, and I do wonder if Rails 5.1 also commits this mistake or not?

[gauravtiwari]

@Gargron Yep it's there too - 5.1+

[wagoid]

Since they check if the yarn:install task is defined before running it (here), I created this same task with an empty body, as a workaround.

namespace :yarn do
  task :install do
    # workaround to stop running yarn install after precompile
  end
end

[noelrappin]

@wagoid Hmm.. for me that runs both my workaround task and the existing task...

[gutierlf]

@wagoid, I also tried your workaround, only to find that it still runs the existing task. 🙁 I'm glad you offered an attempt though. 👍

So does anyone have another workaround? I'm stuck.

Thanks!

[wagoid]

Yeah, I thought that worked at first but it didn't :(
I also tried clearing the rake task, no success