-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
chore(deps): update all dependencies and use webpack 5 #2858
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
Conversation
68c2131
to
8fff82c
Compare
Codecov Report
@@ Coverage Diff @@
## v4 #2858 +/- ##
==========================================
- Coverage 92.62% 92.46% -0.17%
==========================================
Files 38 38
Lines 1247 1247
Branches 326 326
==========================================
- Hits 1155 1153 -2
- Misses 87 89 +2
Partials 5 5
Continue to review full report at Codecov.
|
@@ -4,7 +4,9 @@ const SockJS = require('sockjs-client/dist/sockjs'); | |||
const { log } = require('../default/utils/log'); | |||
const BaseClient = require('./BaseClient'); | |||
|
|||
module.exports = class SockJSClient extends BaseClient { | |||
module.exports = class SockJSClient extends ( |
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.
Cause by editor format?
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.
prettier
was updated to improve formatting for assignments of classes with long superclass names.
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.
Oh, thanks, weird for this to extra brackets, maybe fix by prettier/prettier#9744
issue link prettier/prettier#9341 (comment)
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.
Is not important for this pr
Big thanks |
For Bugs and Features; did you add new tests?
N/A
Motivation / Use-Case
prettier
: improve formatting for assignments of classes with long superclass nameswebpack-merge
: merge has been moved as a regular importschema-utils
: the packages exports was changedpuppeteer
:waitFor
is deprecated (Proposal: deprecatewaitFor
and addwaitForTimeout
puppeteer/puppeteer#6214)target: ['web', 'es5']
for client sourcesBreaking Changes
N/A
Additional Info