Skip to content
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

chore: skip service worker plugin for non PWAs #10361

Merged
merged 1 commit into from
Mar 24, 2021

Conversation

haijian-vaadin
Copy link
Contributor

@haijian-vaadin haijian-vaadin commented Mar 22, 2021

Reduce frontend build time for non-PWA applications by not running the service worker Webpack plugin.

@haijian-vaadin haijian-vaadin added the hilla Issues related to Hilla label Mar 22, 2021
@haijian-vaadin haijian-vaadin force-pushed the haijian/frontend-build-optimization branch from 1480105 to c826076 Compare March 22, 2021 13:49
@haijian-vaadin haijian-vaadin force-pushed the haijian/frontend-build-optimization branch from c826076 to 39f24f7 Compare March 22, 2021 14:39
@haijian-vaadin haijian-vaadin changed the title chore: skip InjectManifestPlugin for non PWAs chore: skip service worker plugin for non PWAs Mar 22, 2021
@haijian-vaadin haijian-vaadin force-pushed the haijian/frontend-build-optimization branch from 39f24f7 to 03332bc Compare March 22, 2021 15:19
joheriks
joheriks previously approved these changes Mar 22, 2021
@haijian-vaadin haijian-vaadin force-pushed the haijian/frontend-build-optimization branch from 03332bc to 8f93bde Compare March 23, 2021 08:00
@haijian-vaadin haijian-vaadin requested a review from pleku March 23, 2021 10:55
@joheriks joheriks self-requested a review March 24, 2021 08:20
@joheriks joheriks merged commit 722bd3b into master Mar 24, 2021
@joheriks joheriks deleted the haijian/frontend-build-optimization branch March 24, 2021 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hilla Issues related to Hilla +0.0.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants