ericclemmons / start-server-webpack-plugin

Automatically start your server once Webpack's build completes.
MIT License
158 stars 26 forks source link

support webpack 5, add logging configuration, add integrationtests and use github actions, correct branch #38

Open fivethreeo opened 4 years ago

fivethreeo commented 4 years ago

@wmertens: What about a silent option? Just not allow silencing since it makes little sense in dev? Maybe if verbose is false and NODE_ENV is test the server and info stays silent, but that could be a issue with other projects running tests with start-server-webpack-plugin. Alternatively subclass the plugin in our tests to make it silent?

fivethreeo commented 4 years ago

Went with subclassing the plugin in our tests to make it silent.

fivethreeo commented 4 years ago

I did a temporary private release just to do some testing of the webpack5 changes. Waiting eagerly for feedback. 😀If you request changes I can do them. But from what I see when using this in razzle a more silent running makes sense. So I am thinking along the lines of a info true/false or using loglevels of some kind and a worker_verbose true/false or document overriding plugin functions.

wmertens commented 4 years ago

I'll try to review this week. In my changes I was trying to be quiet by default, but still output whenever there were issues that impact the user, like the build failing.

fivethreeo commented 4 years ago

@wmertens ping 😀I am using this code successfully in razzle using a test matrix, so to works well. But could you do a alpha/beta release so I don’t have to do my own release and maybe other people can see the new release and try it out? If you want any more changes, just say and I’ll make the changes. 😀

wmertens commented 4 years ago

Okay will do 😊

Wout.

On Sat., Apr. 18, 2020, 1:40 a.m. Øyvind Saltvik notifications@github.com wrote:

@wmertens https://github.com/wmertens ping 😀I am using this code successfully in razzle using a test matrix, so to works well. But could you do a alpha/beta release so I don’t have to do my own release and maybe other people can see the new release and try it out? If you want any more changes, just say and I’ll make the changes. 😀

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ericclemmons/start-server-webpack-plugin/pull/38#issuecomment-615510951, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAANNFSGMTWATAYZL3PUNJLRNDSG3ANCNFSM4LSXXMRA .

RyanEwen commented 4 years ago

Might not be the best place to ask but could you add a feature to auto-restart the server when not using HMR?

fivethreeo commented 4 years ago

Open a new issue for this, also not sure what you mean by that.

RyanEwen commented 4 years ago

https://github.com/ericclemmons/start-server-webpack-plugin/issues/28

It just didn't seem like the original repo issues were getting updates anymore

felixmosh commented 3 years ago

When this PR will be merged? Webpack 5 final version just released :]

fivethreeo commented 3 years ago

Released as razzle-start-server-webpack-plugin now, so code is now at https://github.com/jaredpalmer/razzle

datner commented 3 years ago

Is there any progress on this? The last comment is from October

fivethreeo commented 3 years ago

Just use razzle-start-server-webpack-plugin@canary

xfournet commented 3 years ago

I used "run-script-webpack-plugin": "0.0.5" as replacement since this projet seems to be dead

datner commented 3 years ago

It's working out just fine, I have hmr up and everything, but this should still be implemented in the original lib methinks