Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use bower:update instead of bower:install before assets:precompile #181

Open
mbajur opened this issue Dec 15, 2015 · 0 comments
Open

Use bower:update instead of bower:install before assets:precompile #181

mbajur opened this issue Dec 15, 2015 · 0 comments

Comments

@mbajur
Copy link

mbajur commented Dec 15, 2015

We've had a lot of troubles with bower-rails (wrong asset versions installed on production) and after doing a lot of research, one of our frontend guys explained us what might be wrong with our setup.

I'm using install_before_precompile setting to install all my assets before precompilation on production servers. While in most cases it works as it should, for some reason bower install does not checks (resolution) versions of packages installed. It means that if i have a resolution 'angular', '1.3.16' line - when i do run bower:install, change version number to 2.0 and run bower:install again, bower does not checks what version is installed, it just checks if it's installed at all and does nothing if it is. So the package is not getting updated to 2.0, it's still 1.3.16.

That's why i think it would be a good idea to replace install_before_precompile with update_before_precompile (or map that config to update instead of install) because bower update takes care of resolution changes and also, if there was no bower install triggered before, it runs it.

However, i'm not a bower specialist nor even a big javascript user so i might be wrong all the way and the issue is somewhere else.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant