-
-
Notifications
You must be signed in to change notification settings - Fork 447
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
Rails default Dockerfile build failed after added bootstrap gem #277
Comments
Just like the error message says, you do need some execjs runtime. |
but what is the point of adding support for dartsass-rails if we are still with "nodejs"? The actual usage with importmaps is broken, we still need to install some js runtime as well |
The Sass engine is orthogonal to autoprefixer. The previous Sass engine didn't use nodejs either (sassc is written in C++), it's autoprefixer that does. |
I see, I didn't know that. But in any case it turns out that there is no sense in flow with importmaps? Why keep nodejs and importmaps together in the project? |
I'm not sure what importmaps have to do with any of this |
I've also encountered this issue while trying to build the Dockerfile. Is there a way to make the autoprefixer gem optional? |
I agree, this makes no sense at all. The whole point of going with the rails default importmap setup was the fact that we do not need to install a js related runtime like node anymore in order to run rails. autoprefixer dependency in bootstraps just defeats that purposes isnt it? |
I'm considering making the |
@glebm are you able to give a quick run down why autoprefixer-rails as a dependency for bootstrap is required by rails app? so what happens if this is now something optional? I'm still new to this side of rails and just trying to understand how does this work. Greatly appreciate any form of information from you! |
Any news on release of #283 ? |
rails new test_app cd ../test_app ./bin/bundle add dartsass-rails ./bin/rails dartsass:install
docker build .
Now let's produce error:
./bin/bundle add bootstrap --git 'https://github.com/twbs/bootstrap-rubygem'
docker build .
throw error:
Installing node/etc is not the right solution because the main reason to use importmap is to drop the nodejs dependency. Any ideas on how to solve this?
The text was updated successfully, but these errors were encountered: