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

Allow spdy to be used as webpack dev server. #2071

Merged

Conversation

Hyperkid123
Copy link
Contributor

@Hyperkid123 Hyperkid123 commented Oct 1, 2024

Changes

  • update webpack and its dependencies to latest versions
  • enable setting the _unstableSpdy option from fec.config.js to enable spdy as a dev server

@Hyperkid123 Hyperkid123 added release minor Once merged it will trigger minor release dependencies Pull requests that update a dependency file labels Oct 1, 2024
@Hyperkid123 Hyperkid123 requested a review from a team October 1, 2024 13:11
@Hyperkid123 Hyperkid123 force-pushed the enable-spdy-devserver-config branch from 9eef74d to 1a33d4b Compare October 1, 2024 13:17
@Hyperkid123 Hyperkid123 marked this pull request as ready for review October 1, 2024 13:17
@@ -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';
Copy link
Member

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.

Copy link
Contributor Author

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.

Copy link
Member

@bastilian bastilian left a 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!

@Hyperkid123 Hyperkid123 force-pushed the enable-spdy-devserver-config branch from 1a33d4b to 9efefcc Compare October 2, 2024 07:23
@Hyperkid123 Hyperkid123 merged commit c4d592c into RedHatInsights:master Oct 2, 2024
1 check passed
@Hyperkid123 Hyperkid123 deleted the enable-spdy-devserver-config branch October 2, 2024 07:50
@nacho-bot
Copy link
Collaborator

                      :soon::shipit::octocat:
     :rose:Shipit Squirrel has this release minor surrounded, be ready for a new version:sunflower:

@nacho-bot
Copy link
Collaborator

     🌱 🌸 🌷 🌻 🌟 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🚀

@nacho-bot nacho-bot added the released Pr has been released label Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file release minor Once merged it will trigger minor release released Pr has been released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants