Skip to content

Conversation

bseib
Copy link
Contributor

@bseib bseib commented Sep 30, 2019

Tell users of DevServer.writeToDisk where to find the output that gets written to disk.

fixes #3304

@netlify
Copy link

netlify bot commented Sep 30, 2019

Preview is ready

Built with commit bb0a922

https://deploy-preview-3305--webpackjsorg-netlify.netlify.com

Copy link
Member

@EugeneHlushko EugeneHlushko left a comment

Choose a reason for hiding this comment

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

Looks good but i would suggest removing default of the linked config prop.

`boolean = false` `function (filePath)`

Tells `devServer` to write generated assets to the disk.
Tells `devServer` to write generated assets to the disk. The output is written to the [output.path](/configuration/output/#outputpath) directory, which is `./dist` by default.
Copy link
Member

Choose a reason for hiding this comment

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

Please avoid duplicating defaults, we already have a link.

Suggested change
Tells `devServer` to write generated assets to the disk. The output is written to the [output.path](/configuration/output/#outputpath) directory, which is `./dist` by default.
Tells `devServer` to write generated assets to the disk. The output is written to the [output.path](/configuration/output/#outputpath) directory.

The link to where the default is documented is sufficient.
@EugeneHlushko EugeneHlushko merged commit e5a2d9c into webpack:master Oct 6, 2019
@EugeneHlushko
Copy link
Member

Thanks!

@bseib bseib deleted the 3304-document-writeToDisk-output-dir branch October 8, 2019 20:16
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.

Document the output directory for devServer.writeToDisk
2 participants