-
-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Updates definition build script for babel 7.0 #5024
Conversation
Codecov Report
@@ Coverage Diff @@
## master #5024 +/- ##
==========================================
- Coverage 94.24% 94.17% -0.07%
==========================================
Files 121 122 +1
Lines 8820 9080 +260
==========================================
+ Hits 8312 8551 +239
- Misses 508 529 +21
Continue to review full report at Codecov.
|
@@ -107,8 +107,7 @@ | |||
"lint-staged": { | |||
"{src,spec}/**/*.js": [ | |||
"prettier --write", | |||
"flow", |
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.
if you're removing this, should @flow's come out too?
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.
Yeah, flow doesn’t take an argument, i’ll Find a way to add it back
This reverts commit 617e840.
This reverts commit 617e840.
* Updates definition build script for babel 7.0 * run eslint on staged files
Updating
@babel/core
to 7.0 has introduced a malfunction into the buildConfigDefinitions script that this PR addresses.