Skip to content

Revert "Bump eventsource, sockjs-client and webpack-dev-server in /openhtf/output/web_gui"#1059

Merged
glados-verma merged 1 commit intomasterfrom
revert-1026-dependabot/npm_and_yarn/openhtf/output/web_gui/eventsource-and-sockjs-client-and-webpack-dev-server-2.0.2
Oct 13, 2022
Merged

Revert "Bump eventsource, sockjs-client and webpack-dev-server in /openhtf/output/web_gui"#1059
glados-verma merged 1 commit intomasterfrom
revert-1026-dependabot/npm_and_yarn/openhtf/output/web_gui/eventsource-and-sockjs-client-and-webpack-dev-server-2.0.2

Conversation

@glados-verma
Copy link
Collaborator

@glados-verma glados-verma commented Oct 13, 2022

Reverts #1026

webpack v3 to v4 is a serious migration, we can't just bump the version number. Right now, running node run start for web_gui fails because webpack-cli isn't installed; one cannot be installed without bumping webpack, and webpack cannot be bumped without non-trivial changes to at least the webpack config.

Rolling back for now, we'll take a look at this later.


This change is Reviewable

@dbhatman dbhatman self-requested a review October 13, 2022 23:40
@glados-verma
Copy link
Collaborator Author

Verified that npm run start works again.

@glados-verma glados-verma merged commit d1006c3 into master Oct 13, 2022
@glados-verma glados-verma deleted the revert-1026-dependabot/npm_and_yarn/openhtf/output/web_gui/eventsource-and-sockjs-client-and-webpack-dev-server-2.0.2 branch October 13, 2022 23:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants