-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
generate css files in build to use with csp #4558
Conversation
9a7c769
to
08ef359
Compare
Codecov Report
@@ Coverage Diff @@
## master #4558 +/- ##
==========================================
+ Coverage 70.93% 71.05% +0.12%
==========================================
Files 544 546 +2
Lines 55209 55279 +70
Branches 10330 10340 +10
==========================================
+ Hits 39161 39280 +119
+ Misses 16048 15999 -49
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Makefile.dryice.js
Outdated
|
||
if (callback) | ||
return callback(); | ||
console.log("Finished building " + getTargetDir(options)); | ||
} | ||
} | ||
|
||
function extractCss(options) { |
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.
Is that ok that we don't await on creation of theme files here, and on extractCss
in general? Feels like extractCss
should accept a callback to, but I don't really have much context here.
Hello, are there any updates for the CSP topic? |
6721908
to
e1d156b
Compare
Hi, CSP changes are released in v1.4.13 today. |
see #4554 for context
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.