-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
refactor: the hot option
#2409
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
refactor: the hot option
#2409
Conversation
BREAKING CHANGE: the `hotOnly` option was remove, please use `hot: "only"` instead
Codecov Report
@@ 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
Continue to review full report at Codecov.
|
| }); | ||
|
|
||
| it("should prepends webpack's hot-only client script", () => { | ||
| it("should prepends webpack's hot only client script", () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| it("should prepends webpack's hot only client script", () => { | |
| it("should prepend webpack's hot only client script", () => { |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep
For Bugs and Features; did you add new tests?
using existing
Motivation / Use-Case
Simplify configuration
Breaking Changes
Yes
Additional Info
No