-
Notifications
You must be signed in to change notification settings - Fork 160
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
option to inline external dependencies? #26
Comments
It's not a bad idea if you really need to keep everything to one file. Inlining the JS/CSS shouldn't pose too much of an issue. The fonts would need to be converted to base64 and then included directly into the CSS. The resulting all-in-one file might be pretty big. If you want to work up a PR I'll definitely take a look. |
@glenjamin I had some time today to look into this. This PR should do what you need: #27 |
@adamgruber Just curious, was this implemented? I looked at the latest build readme and don't see any mention of this capability? If so, how to enable it, think this could fix my issue as well |
@adamgruber forget the above, I have found the appropriate property! |
Let me know if it works for you On Tue, Jan 19, 2016 at 10:16 PM Mike Erickson notifications@github.com
|
Hello!
This report looks ace, but I'd quite like a way to bundle everything into a single file - rather than having to zip the report up to move it around.
Is that something you'd be open to exposing as an option?
I can probably provide a PR if you're on board with the idea.
The text was updated successfully, but these errors were encountered: