-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Babel 7 #3746
Babel 7 #3746
Changes from 73 commits
c704d60
69cc530
cb988d0
a4f9a2e
084671c
113c182
73dc63b
b6153d1
0f93c5e
ee5c7c7
720bb23
e496dfb
87312a6
a050bcb
74475b9
c4e468a
ee6472d
170160d
3c0e383
6da0f7a
4b169ab
ba544ef
d0b7972
e0ad03d
a87d16d
0ec2bc0
6371e1f
a039c96
6e2d8c1
a8e41c9
fe86390
e6c055c
bf3c2e5
b649f24
f7cd337
9a1abf2
3ae34e5
850b083
1209703
7ed57a8
04f6ef8
3c027a4
41d6168
29e1c02
107f5e0
f627592
2b4f05b
f36869a
df723e0
61d5c09
224e1f9
aadd8f0
4fcf8cb
3dec3ba
5b9226b
c6dd3e4
773821d
bfccae4
bbf7d8a
30fe991
7b5dc84
40274f9
165e996
cb69d42
305fff1
f8332e7
c0cd951
7102951
451b8af
a05565b
72e4b84
ad78be3
70d9094
2028cd4
94a049c
f1cf6d2
9c4cff4
9648d20
bb3e725
961fdef
36a2676
489d55d
06d799c
c5f5433
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,16 +1,33 @@ | ||
{ | ||
"presets": ["env", "stage-0", "react"], | ||
"env": { | ||
"test": { | ||
"plugins": ["require-context-hook"] | ||
}, | ||
"plugins": [ | ||
"emotion", | ||
"babel-plugin-macros", | ||
["transform-runtime", { | ||
"polyfill": false, | ||
"presets": [ | ||
"@babel/preset-env", | ||
"@babel/preset-react", | ||
"@babel/preset-flow" | ||
], | ||
"plugins": [ | ||
"babel-plugin-emotion", | ||
"babel-plugin-macros", | ||
"@babel/plugin-proposal-class-properties", | ||
"@babel/plugin-proposal-export-default-from", | ||
[ | ||
"@babel/plugin-transform-runtime", | ||
{ | ||
"regenerator": true | ||
}] | ||
} | ||
] | ||
} | ||
], | ||
"env": { | ||
"test": { | ||
"plugins": ["babel-plugin-require-context-hook"] | ||
} | ||
}, | ||
"overrides": [ | ||
{ | ||
"test": "./examples/vue-kitchen-sink", | ||
"presets": [ | ||
"@babel/preset-env", | ||
"babel-preset-vue" | ||
] | ||
} | ||
] | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -23,7 +23,6 @@ | |
"@storybook/addons": "4.0.0-alpha.16", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Runtime package isn't actually needed here, it's in resolve.alias |
||
"@storybook/components": "4.0.0-alpha.16", | ||
"@storybook/core-events": "4.0.0-alpha.16", | ||
"babel-runtime": "^6.26.0", | ||
"deep-equal": "^1.0.1", | ||
"emotion-theming": "^9.2.6", | ||
"global": "^4.3.2", | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,8 +1,6 @@ | ||
import PropTypes from 'prop-types'; | ||
import React from 'react'; | ||
|
||
// TODO: FIX this dependency cycle | ||
/* eslint-disable import/no-cycle */ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ESLint packages update seems quite irrelevant to this PR, and it makes it harder to review =( |
||
import Shape from './Shape'; | ||
import OneOfType from './OneOfType'; | ||
import ArrayOf from './ArrayOf'; | ||
|
@@ -11,7 +9,6 @@ import OneOf from './OneOf'; | |
import InstanceOf from './InstanceOf'; | ||
import Signature from './Signature'; | ||
import Literal from './Literal'; | ||
/* eslint-enable import/no-cycle */ | ||
|
||
import { TypeInfo } from './proptypes'; | ||
|
||
|
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 there anything left that it still disallows?
BTW this rule is about performance so we probably shouldn't shut it up like this
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.
Shall we restrict this after merger and fix issues that come up then?
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.
Sounds good