-
Notifications
You must be signed in to change notification settings - Fork 508
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
minify service worker script before html inlining (#59) #61
minify service worker script before html inlining (#59) #61
Conversation
de50b6a
to
6432e80
Compare
6432e80
to
990f1ea
Compare
@addyosmani Can you please take a look at this? |
@anubhav7495 Btw, just in case you might be interested in helping out with this repo too I've added you as a collaborator to it :) |
@@ -10,6 +10,7 @@ var HtmlWebpackPlugin = require('html-webpack-plugin') | |||
var ExtractTextPlugin = require('extract-text-webpack-plugin') | |||
var OptimizeCSSPlugin = require('optimize-css-assets-webpack-plugin') | |||
var SWPrecacheWebpackPlugin = require('sw-precache-webpack-plugin') | |||
var loadMinified = require('./load-minfied') |
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.
@anubhav7495 Typo here ☝️
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.
Thanks. I will push a fix immediately.
@addyosmani Thanks. I have merged this PR because |
Cheers, @anubhav7495! |
I have added the solution to minify the
service-worker-prod.js
before inlining in index.html.The approach works for me in production here: hnpwa-vue
Requesting reviews.
Thanks.