Skip to content

Conversation

@alexander-akait
Copy link
Member

  • This is a bugfix
  • This is a feature
  • This is a code refactor
  • This is a test update
  • This is a docs update
  • This is a metadata update

For Bugs and Features; did you add new tests?

using existing

Motivation / Use-Case

Simplify configuration

Breaking Changes

Yes

Additional Info

No

BREAKING CHANGE: the `hotOnly` option was remove, please use `hot: "only"` instead
@codecov
Copy link

codecov bot commented Jan 29, 2020

Codecov Report

Merging #2409 into next will decrease coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##             next    #2409      +/-   ##
==========================================
- Coverage   91.99%   91.98%   -0.02%     
==========================================
  Files          36       36              
  Lines        1324     1322       -2     
  Branches      373      372       -1     
==========================================
- Hits         1218     1216       -2     
  Misses         97       97              
  Partials        9        9
Impacted Files Coverage Δ
lib/Server.js 91.93% <100%> (ø) ⬆️
lib/utils/createConfig.js 98.16% <100%> (-0.04%) ⬇️
lib/utils/updateCompiler.js 100% <100%> (ø) ⬆️
lib/utils/addEntries.js 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 221bba4...49194c7. Read the comment docs.

});

it("should prepends webpack's hot-only client script", () => {
it("should prepends webpack's hot only client script", () => {
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
it("should prepends webpack's hot only client script", () => {
it("should prepend webpack's hot only client script", () => {

Copy link
Member Author

Choose a reason for hiding this comment

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

I found a lot of mistakes in test names, let's open issue about it and fix all of them in a separate PR

Copy link
Member

Choose a reason for hiding this comment

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

Shall I take it up?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yep

@alexander-akait alexander-akait merged commit 036fa17 into next Jan 29, 2020
@alexander-akait alexander-akait deleted the refactor-hot branch January 29, 2020 17:09
@jamesgeorge007 jamesgeorge007 mentioned this pull request Jan 29, 2020
6 tasks
alexander-akait pushed a commit that referenced this pull request Jan 31, 2020
BREAKING CHANGE: the `hotOnly` option was remove, please use `hot: "only"` instead
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.

3 participants