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

docs(config) document devServer options injectClient and injectHot #2996

Merged
merged 4 commits into from
May 24, 2019

Conversation

EugeneHlushko
Copy link
Member

@EugeneHlushko EugeneHlushko commented May 5, 2019

Takes two off #2974
Documents devServer options injectClient and injectHot

ON HOLD due to waiting dev server release.

@netlify
Copy link

netlify bot commented May 5, 2019

Preview is ready

Built with commit ef09119

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

module.exports = {
//...
devServer: {
injectHot: (compilerConfig) => compilerConfig.name === 'only-include'
Copy link
Member

Choose a reason for hiding this comment

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

Please add hot because injectHot is used when HMR is enabled.

Copy link
Member Author

Choose a reason for hiding this comment

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

Great catch @hiroppy
I've pushed an update

};
```

W> Make sure that [`devServer.hot`](#devserverhot) is set to `true` because `devServer.injectHot` only works with HMR.
Copy link
Member

Choose a reason for hiding this comment

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

W> ?

Copy link
Member Author

Choose a reason for hiding this comment

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

You can see it in preview mate, or check out https://webpack.js.org/contribute/writers-guide

Copy link
Member

Choose a reason for hiding this comment

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

Oops, thank you for the explanation.

@hiroppy
Copy link
Member

hiroppy commented May 17, 2019

We have released [email protected], so I remove on hold label. /cc @EugeneHlushko

@hiroppy hiroppy removed the On hold label May 17, 2019
@EugeneHlushko
Copy link
Member Author

Amazing, ty @hiroppy

@EugeneHlushko EugeneHlushko merged commit 4874375 into master May 24, 2019
@EugeneHlushko EugeneHlushko deleted the docs/dev-server-injects branch May 24, 2019 08:31
@EugeneHlushko
Copy link
Member Author

Thanks everyone invlolved

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.

4 participants