-
Notifications
You must be signed in to change notification settings - Fork 24.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
Move and Rename propType definitions in ShadowPropTypesIOS #21379
Conversation
Looks like you have a merge conflict. ColorPropType was changed to DeprecatedColorPropType in another commit. |
@TheSavior, just updated the PR. Sorry about that. 🤖 |
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.
TheSavior is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
No problem, not your fault. Just a race condition from landing the other one first. Thanks for updating and thanks for contributing! |
@gvarandas merged commit f94d2ad into Once this commit is added to a release, you will see the corresponding version tag below the description at f94d2ad. If the commit has a single |
…21379) Summary: related facebook#21342 move TransformPropTypes.js fix flow error - [x] yarn prettier - [x] yarn flow-check-android - [x] yarn flow-check-ios All flow checks pass. [GENERAL] [ENHANCEMENT] [DeprecatedShadowPropTypesIOS.js] - Created. Pull Request resolved: facebook#21379 Differential Revision: D10098750 Pulled By: TheSavior fbshipit-source-id: f7f2e4bf7b837c00a14b1fbd930d1b29ffb63549
related #21342
TODO
move TransformPropTypes.js
fix flow error
CheckList
Test Plan
All flow checks pass.
Release Notes
[GENERAL] [ENHANCEMENT] [DeprecatedShadowPropTypesIOS.js] - Created.