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: code #482
refactor: code #482
Conversation
After this PR i will start rewriting logger on webpack logger |
Codecov Report
@@ Coverage Diff @@
## master #482 +/- ##
=======================================
Coverage 97.21% 97.21%
=======================================
Files 13 13
Lines 323 323
Branches 99 99
=======================================
Hits 314 314
Misses 9 9
Continue to review full report at Codecov.
|
.eslintignore
Outdated
/coverage | ||
/dist | ||
/node_modules | ||
/test/fixtures |
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 the line break?
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.
fixed
.prettierignore
Outdated
/dist | ||
/node_modules | ||
/test/fixtures | ||
CHANGELOG.md |
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.
ditto
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.
fixed
"test:watch": "npm run test:only --watch", | ||
"test:only": "cross-env NODE_ENV=test jest", | ||
"test:watch": "cross-env NODE_ENV=test jest --watch", | ||
"test:coverage": "cross-env NODE_ENV=test 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.
It is better to separate to jest.config.js?
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 it is more simple than using jest.config.js
. Maybe we revisit this in future.
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.
ok
31723aa
to
ba30e47
Compare
This PR contains a:
Motivation / Use-Case
index.js
inside lib (let's do not mix configuration files in root with code)'use strict'
in some placesBreaking Changes
No
Additional Info
No