-
Notifications
You must be signed in to change notification settings - Fork 19
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
Deps fix #19
Conversation
package.json
Outdated
"webpack-cli": "^3.2.1" | ||
}, | ||
"lint-staged": { | ||
"src/**/*.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.
you can use {src,test}/**/*.js
also prettier --write
should be enough
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.
done, thanks :)
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.
@ForsakenHarmony i had to re-open these because tests have fixtures/**/*.js
and i din't knwo a good way of excluding them in this single statement
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.
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.
gotcha thanks :), u da best
"ava": "^1.0.1", | ||
"deepcopy": "^1.0.0", | ||
"webpack": "^4.28.4" | ||
"chalk": "^2.4.1", |
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.
not kleur 🙃
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 yes probably in next PR then
Oh I'm so sorry, I completely missed the notification for this as well as you asking me for a PR 😥 |
package.json looks good though 👍 thanks for the quick fix |
fixes #16
@jgierer12 can you please review
package.json
?