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

[Remove Node.js] Use dart-sass for SCSS #523

Open
1 of 3 tasks
malparty opened this issue Jul 9, 2024 · 1 comment
Open
1 of 3 tasks

[Remove Node.js] Use dart-sass for SCSS #523

malparty opened this issue Jul 9, 2024 · 1 comment

Comments

@malparty
Copy link
Member

malparty commented Jul 9, 2024

inherit #431

Note

This issue does NOT include any code change in the Rails template (yet), as these will be based on the RFC results.

Why

SCSS build currently relies on Node.js.

Checklist

Insight 📝

The cssbundling-rails gem relies on Node.js.
[Rails Guides](https://guides.rubyonrails.org/asset_pipeline.html#[dartsass-rails](https://guides.rubyonrails.org/asset_pipeline.html#dartsass-rails)) recommend using the dartsass-rails to remove Node.js dependency.

./bin/bundle add dartsass-rails
./bin/rails dartsass:install

Note

It turns out import-maps is limited to JS and won’t let you pin other assets like SCSS or images.
rails/importmap-rails#80

If you need Node for anything, I'd use it for everything. The benefit of import maps is that it provides a path with no Node at all. cssbundling-rails and jsbundling-rails is a great combo to use with Bootstrap 👍”
DHH – [source](rails/importmap-rails#107 (comment))

Who Benefits?

  • Developers for the knowledge
  • Future clients' projects for improved tooling decisions.
@malparty
Copy link
Member Author

malparty commented Jul 9, 2024

Issue started in https://github.com/nimblehq/cx-web/pull/757

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

No branches or pull requests

1 participant