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
Compress Website Sass output #6009
Conversation
rake/site.rake
Outdated
@@ -47,7 +47,7 @@ namespace :site do | |||
task :update_normalize_css do | |||
Dir.chdir("#{docs_folder}/_sass") do | |||
sh 'curl "https://necolas.github.io/normalize.css/latest/normalize.css" -o "normalize.scss"' | |||
sh 'sass "normalize.scss":"_normalize.scss" --style compressed' | |||
sh 'sass "normalize.scss":"_normalize.scss"' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the purpose of this?
Why not:
sh 'curl "https://necolas.github.io/normalize.css/latest/normalize.css" -o "_normalize.scss"'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems that previously we used a Sass workflow, and now we rely entirely on GitHub Pages pipeline. We therefore can simplify here to the bare minimum, you're right.
Suggestion applied. 🖖
Just copy the latest version to the Website Sass folder and let Jekyll handle the compression. props @pathawks
…as long as there isn't some reason I'm not thinking of to not do this |
This is automatically done in production - are you seeing different results on jekyllrb.com? |
@parkr https://jekyllrb.com/css/screen.css does not look compressed to me |
You're absolutely right! It defaults to |
@jekyllbot: merge +docs |
Suggested by @pathawks in #6008