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

Backport support for configuration as Promise in Webpack v1 #698

Merged
merged 1 commit into from
Jan 31, 2017

Conversation

mathieumg
Copy link

@mathieumg mathieumg commented Nov 16, 2016

What kind of change does this PR introduce?

  • Bugfix

Does this PR introduce a breaking change?

  • No

Other information:

Backports the fix in #419 to webpack-dev-server v1 since v1 of Webpack itself supports config as Promise. Change looks more complex than it really is, it's just because a lot of code is getting wrapped in a function.

@jsf-clabot
Copy link

jsf-clabot commented Nov 16, 2016

CLA assistant check
All committers have signed the CLA.

@mathieumg
Copy link
Author

@SpaceK33z It's been a month today, I don't mean to be impatient but would it be possible to take a look at this? It's essentially a two-liner that is already in Webpack 1, just need to have it in the dev server as well. Thanks!

@SpaceK33z
Copy link
Member

@mathieumg Honestly I wasn't planning on doing another v1 release, since v2 is right around the corner and now we're in release candidate already. However, if I have the time this weekend I'll see if I can do another v1 release.

@mathieumg
Copy link
Author

Cool, thanks for the information. That would be appreciated to "close the book" on v1. Like I said, it's cool that the Promise config was backported from v2 to v1, it's just impractical to use it since it wasn't backported to the dev server at the same time. Thanks!

@mathieumg
Copy link
Author

@SpaceK33z Sorry to bother you, do you think it would be possible to merge this? It seems to be the last v1 PR still open. Thanks a lot!

@SpaceK33z
Copy link
Member

No, I'm sorry. I have been closing webpack 1 specific PR's for a while. webpack 2 is released now and we'll only focus on critical bugfixes in v1.

@SpaceK33z
Copy link
Member

Oh I kinda forgot that this was a bugfix not a feature. I'll see if I have some time later today.

@SpaceK33z SpaceK33z merged commit 700b19d into webpack:webpack-1 Jan 31, 2017
@mathieumg mathieumg deleted the bugfix_config-promise-v1 branch January 31, 2017 22:17
@mathieumg
Copy link
Author

Thank you! 🎉

@SpaceK33z
Copy link
Member

np, released as webpack-dev-server@1.16.3. Sorry for the long waiting time.

@richtera
Copy link

richtera commented Mar 19, 2017

👍 Works. Now I can use portscanner to search for the first free port :)

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.

4 participants