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
Improve asset compilation in Builder UI #3905
Conversation
This change makes a few minor improvements to asset compilation in Builder: * Removes CSS linting from compilation (we have a separate task for that) * Builds CSS once before watching in the development environment (as passing `—watch` suppresses initial compilation) * Removes CSS and JS source maps from production builds * Single quotes in the Webpack config Signed-off-by: Christian Nunciato <cnunciato@chef.io>
Thanks for the pull request! Here is what will happen next:
Thank you for contributing! |
@@ -1,6 +1,8 @@ | |||
#!/bin/sh | |||
set -x | |||
|
|||
npm run build-css -- --source-map=true |
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 reasoning for source-map=true here? Since we are only doing compression in production, do we need source mapping at all now?
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.
This script is only run in dev, and yeah, we do use it in dev.
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.
Ok cool
@thesentinels approve |
🤘 I am testing your branch against master before merging it. We do this to ensure that the master branch is never failing tests. |
Travis CI has started testing this PR. |
💖 Travis CI reports this PR passed. It always makes me feel nice when humans approve of one anothers work. I'm merging this PR now. I just want you and the contributor to answer me one question: |
This change makes a few minor improvements to asset compilation in Builder:
--watch
suppresses initial compilation)Signed-off-by: Christian Nunciato cnunciato@chef.io
Fixes #3526.