Skip to content

docs(devServer): add sockPath docs - #2791

Merged
EugeneHlushko merged 6 commits into
webpack:masterfrom
trescenzi:add-sockpath-docs
Mar 25, 2019
Merged

docs(devServer): add sockPath docs#2791
EugeneHlushko merged 6 commits into
webpack:masterfrom
trescenzi:add-sockpath-docs

Conversation

@trescenzi

Copy link
Copy Markdown
Contributor

Related PR

This adds documentation for the sockPath option added by the related PR above.

@montogeek

Copy link
Copy Markdown
Contributor

This change is already released?

@trescenzi

Copy link
Copy Markdown
Contributor Author

@montogeek that's a good question. It's been merged but I don't know the release cycle of webpack-dev-server. According to the releases page the answer is, currently, no.

@evilebottnawi what is the release strategy of webpack-dev-server and when might sockPath get released?

@alexander-akait

Copy link
Copy Markdown
Member

@trescenzi in near future

@EugeneHlushko EugeneHlushko left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In that case we cannot merge this in, please ping us on this PR again when its merged

@arusahni

Copy link
Copy Markdown

Hi, this appears to be in 3.2.1.

Comment thread src/content/configuration/dev-server.md Outdated
Comment thread src/content/configuration/dev-server.md Outdated
@EugeneHlushko

Copy link
Copy Markdown
Contributor

@trescenzi are you still around to apply suggestion?

EugeneHlushko and others added 2 commits March 25, 2019 12:17
Co-Authored-By: trescenzi <trescenzi@gmail.com>
Co-Authored-By: trescenzi <trescenzi@gmail.com>
@trescenzi

Copy link
Copy Markdown
Contributor Author

@EugeneHlushko let me know if there's anything else I should change.

@EugeneHlushko
EugeneHlushko merged commit 7ffe2a5 into webpack:master Mar 25, 2019
@EugeneHlushko

Copy link
Copy Markdown
Contributor

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants