-
Notifications
You must be signed in to change notification settings - Fork 24.6k
Do not run packager in Release mode #24929
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
Do not run packager in Release mode #24929
Conversation
@cpojer, @grabbou & @deepsweet: please review. |
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.
Thanks for fixing it!
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.
@cpojer is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Diff got stuck, let me try again. |
This pull request was successfully merged by @lucasbento in 4ea6204. When will my fix make it into a release? | Upcoming Releases |
Summary: This PR fixes a regression introduced on [3273d23](facebook@3273d23#diff-883359f85083d00b7266ec2acebcca9f) that removed facebook#23938. It also adds the check for `React` and `React-tvOS` targets. ## Changelog [IOS] [FIXED] - Do not run packager in Release mode Pull Request resolved: facebook#24929 Differential Revision: D15391692 Pulled By: cpojer fbshipit-source-id: c2e524108be3bf1e45b0d0ff7cddd9785b51a60f
How can I check the fix version. Is this issue on 0.59 and fix in 0.60 |
Summary
This PR fixes a regression introduced on 3273d23 that removed #23938.
It also adds the check for
React
andReact-tvOS
targets.Changelog
[IOS] [FIXED] - Do not run packager in Release mode
Test Plan
Run RNTester in Release mode - packager shouldn't start anymore