-
-
Notifications
You must be signed in to change notification settings - Fork 922
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
[WIP] Use propshaft instead of sprockets #3470
Conversation
for compatibility with img-src CSP
Codecov Report
@@ Coverage Diff @@
## master #3470 +/- ##
=======================================
Coverage 97.80% 97.80%
=======================================
Files 166 166
Lines 4145 4145
=======================================
Hits 4054 4054
Misses 91 91
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
I like the idea of switching to propshaft! I don't really feel like the 5 npm packages we use justify using esbuild. Can we use importmap-rails and skip the entire JS build step, removing esbuild and jsbundling entirely? Related, since we don't seem to actually have any Sass, can we switch to completely vanilla CSS files (with regular-ass @import statements if needed), and completely skip the Sass step? If we do both of those things, we should be able to
which is (in my opinion) the main selling point of propshaft vs sprockets. |
or to use dartsass-rails and importmap-rails? I have no strong opinion about jsbundling-rails/cssbundling-rails vs importmap-rails/dartsass-rails(or propshaft with sass). |
Oh yeah, if we really need Sass we can use dartsass-rails and also skip Node 👍🏻 But if I'm reading this diff right, we don't currently use Sass, so I think we can get away with just plain CSS and no CSS compiler at all. |
The only sass file is https://github.com/rubygems/rubygems.org/blob/master/vendor/assets/stylesheets/github_buttons.css.scss, but that's actually css file if I understand it well. I think the simple assets pipeline, same as generated with On the other side propshaft is new and having dev problems like rails/propshaft#90. 🤔 |
Closing this in favor of #4123. |
No description provided.