-
Notifications
You must be signed in to change notification settings - Fork 78
Conversation
- SentryReactNative (= 0.30.3) | ||
- yoga (= 0.54.4.React) | ||
- SentryReactNative (= 0.38.0) | ||
- yoga (= 0.55.4.React) |
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.
they don't bump the number until the release
and I updated sentry-reactnative also
"tslint": "^5.8.0", | ||
"tslint-config-prettier": "^1.6.0", | ||
"typescript": "^2.5.3", | ||
"typescript-styled-plugin": "^0.1.2" | ||
"typescript": "2.6.2", |
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.
this is the specific version we were on, updating any further starts to break the types in RN and SC
accessible={true} | ||
hitSlop={undefined} | ||
nativeID={undefined} | ||
onLayout={undefined} | ||
onResponderGrant={[Function]} |
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.
looks like snapshots just don't include this stuff now
@@ -132,7 +125,7 @@ ShallowWrapper { | |||
"behavior": "padding", | |||
"children": Array [ | |||
undefined, | |||
<Styled(View) | |||
<Styled(Component) |
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.
likely from the bump to SC-jest
'../src/lib/Scenes/Favorites/__stories__/Favorites.story', | ||
'../src/lib/Scenes/Home/Components/Sales/Components/__stories__/LotsByFollowedArtists.story', | ||
'../src/lib/Scenes/Home/__stories__/Home.story', | ||
'../src/lib/Scenes/Settings/__stories__/Settings.story', | ||
|
||
]; |
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.
sure, why not
deprecated by #1210 |
Every page looks like I'd expect. This did note that we're getting really far behind WRT the react-native typings, it looks tricky to migrate those, so I made that a separate task from actually updating.