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

Remove broken webpack hot reload from server #3989

Merged
merged 1 commit into from Jun 11, 2023
Merged

Conversation

eikhr
Copy link
Member

@eikhr eikhr commented Jun 10, 2023

Description

The main point of this is to remove the ReactRefreshWebpackPlugin that I unnecessarily added to the webpack server config a while back, but I figured I could also remove the hot reloading code from the server as it doesn't seem to be working. I removed instead of fixing it as I don't really see the need to have hot reload on the server, it is rarely changed much.

Result

Removed some code from webpack config and server code

Testing

  • I have thoroughly tested my changes.

Hot reload still works in client-code.

@eikhr eikhr added the chore Pull requests that does something "boring", yet important, e.g. cleaning up code label Jun 10, 2023
@eikhr eikhr requested a review from LudvigHz June 10, 2023 16:48
@github-actions github-actions bot added the review-needed Pull requests that need review label Jun 10, 2023
Copy link
Member

@ivarnakken ivarnakken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure that makes sense

@eikhr eikhr merged commit f19ca3c into master Jun 11, 2023
4 checks passed
@eikhr eikhr deleted the clean-up-webpack branch June 11, 2023 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Pull requests that does something "boring", yet important, e.g. cleaning up code review-needed Pull requests that need review
Projects
None yet
2 participants