-
Notifications
You must be signed in to change notification settings - Fork 102
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
Allow spdy to be used as webpack dev server. #2071
Allow spdy to be used as webpack dev server. #2071
Conversation
9eef74d
to
1a33d4b
Compare
@@ -2,7 +2,7 @@ import { LogType, fecLogger } from '@redhat-cloud-services/frontend-components-c | |||
import { Compiler, WebpackPluginInstance } from 'webpack'; | |||
import { CommonConfigOptions } from './createConfig'; | |||
|
|||
const { SourceMapDevToolPlugin } = require('webpack'); | |||
import { SourceMapDevToolPlugin } from 'webpack'; |
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.
The constructor below was still raising an error when starting webpack. Changing this import to just import webpack form 'webpack'
and the constructor below to new webpack.SourceMapDevToolPlugin(
seemed to solve the issues.
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 let me change it.
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.
LGTM! I only had to change the Sourcemap thingy, maybe that needs some attention still, but I tested Compliance with the _unstableSpdy
option and everything is zooooomin, aka it's fast, so fast! Thank you for looking into it @Hyperkid123!
1a33d4b
to
9efefcc
Compare
:soon::shipit::octocat: |
🌱 🌸 🌷 🌻 🌟 New version of package has been released 🌟 🌻 🌷 🌸 🌱 The release is available on: :package:@redhat-cloud-services/frontend-components-config/v/6.3.0📦 :boom:This feature is brought to you by probot🚀 |
Changes
_unstableSpdy
option from fec.config.js to enable spdy as a dev server