-
-
Notifications
You must be signed in to change notification settings - Fork 361
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
chore(default): migrate #394
Conversation
30ca3d7
to
0ae1816
Compare
Codecov Report
@@ Coverage Diff @@
## master #394 +/- ##
==========================================
+ Coverage 96.86% 97.03% +0.17%
==========================================
Files 13 7 -6
Lines 287 304 +17
Branches 83 103 +20
==========================================
+ Hits 278 295 +17
Misses 9 9
Continue to review full report at Codecov.
|
rules: { | ||
'prettier/prettier': ['error'], | ||
strict: ['error', 'safe'], | ||
strict: ['off'], |
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.
We need migrate on babel in next PR
npm audit works very bad 😞 |
package.json
Outdated
"test": "jest", | ||
"release": "standard-version" | ||
"test:watch": "jest --watch", | ||
"test:coverage": "jest --collectCoverageFrom='lib/**/*.js' --coverage", |
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 want to create jest.config.js
like webpack-dev-server.
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 think we need do same in webpack-default
and here
.github/CONTRIBUTING.md
Outdated
@@ -0,0 +1,136 @@ | |||
# Contributing in @webpack-contrib |
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.
- @webpack-contrib
+ webpack-dev-middleware
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 is org, let's use webpack
@@ -0,0 +1,136 @@ | |||
# Contributing in @webpack-contrib | |||
|
|||
We'd always love contributions to further improve the webpack / webpack-contrib ecosystem! |
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.
- webpack-contrib
+ webpack-dev-middleware
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.
Here means thanks fro contributing and in webpack and in webpack ecosystem
@evilebottnawi Can you add renovate to this repository? |
@hiroppy What we should do here to enable this? |
0ae1816
to
6211696
Compare
"test": "jest", | ||
"release": "standard-version" | ||
"test:coverage": "npm run test:only -- --coverage", | ||
"test:watch": "npm run test:only --watch", |
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.
/cc @hiroppy watch should be without coverage, so better use test:only
(we need fix it in webpack-dev-server)
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.
watch should be without coverage
hm... honestly, I want to be able to see coverage when running as test:watch
.
/c @hiroppy fixed |
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.
Rubber-stamp LGTM, Thanks!
955982b
to
9c36a30
Compare
What kind of change does this PR introduce?
refactoring
Summary
Does this PR introduce a breaking change?
No
Other information
No