Skip to content
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

Add iOS prod flag from RN 0.69.1->0.69.2 #791

Merged
merged 1 commit into from
Jul 20, 2022

Conversation

leotm
Copy link
Owner

@leotm leotm commented Jul 20, 2022

  • missed in 0.69.1->0.70.0-rc.0 upg
  • since 0.69.2 released 4 days later
  • add 0.69.1->0.69.2 upg
  • changes still not in main template
    • branches /main or 0.70-stable

Did
https://react-native-community.github.io/upgrade-helper/?from=0.69.1&to=0.70.0-rc.0

Now need to apply out 4 days later
https://react-native-community.github.io/upgrade-helper/?from=0.69.1&to=0.69.2

Now need to PR to main template

Fix: #757

Unblock: #690

- missed in 0.69.1->0.70.0-rc.0 upg
- since 0.69.2 released 4 days later
- add 0.69.1->0.69.2 upg
- changes still not in main template
  - branches /main or 0.70-stable

Did
https://react-native-community.github.io/upgrade-helper/?from=0.69.1&to=0.70.0-rc.0

Now need to apply out 4 days later
https://react-native-community.github.io/upgrade-helper/?from=0.69.1&to=0.69.2

Now need to PR to main template

Fix: #757
@leotm leotm added ios Native iOS changes (Swift, C++, CocoaPods) flipper Flipper changes cocoapods CocoaPods changes for building iOS and tvOS release mode Production build variant p: Facebook Partner: Meta labels Jul 20, 2022
@leotm leotm marked this pull request as ready for review July 20, 2022 21:07
@leotm leotm merged commit 998ba4e into master Jul 20, 2022
@leotm leotm deleted the rn-0.69.1-0.69.2-ios-prod-flag branch July 20, 2022 21:08
leotm added a commit that referenced this pull request Jul 21, 2022
…flag"

This reverts commit 998ba4e, reversing
changes made to b678e17.
facebook-github-bot pushed a commit to facebook/react-native that referenced this pull request Jul 22, 2022
Summary:
### Mentioned
- pr[main]: #33882
- discussion: reactwg/react-native-releases#21 (reply in thread)
- pr[0.69-stable]: #34098

Close: #33764

Saw the issue ago couple wks too: leotm/react-native-template-new-architecture#757
Fixed similarly: leotm/react-native-template-new-architecture#791

## Changelog

[iOS] [Changed] - Update Podfile to allow `PRODUCTION=1 pod install`

[CATEGORY] [TYPE] - Message

Pull Request resolved: #34234

Test Plan: Everything builds and runs as expected

Reviewed By: cortinico

Differential Revision: D38029117

Pulled By: cipolleschi

fbshipit-source-id: bdb58200a999cb66f1043a2feb670f9037c8e463
kelset pushed a commit to facebook/react-native that referenced this pull request Jul 27, 2022
Summary:
### Mentioned
- pr[main]: #33882
- discussion: reactwg/react-native-releases#21 (reply in thread)
- pr[0.69-stable]: #34098

Close: #33764

Saw the issue ago couple wks too: leotm/react-native-template-new-architecture#757
Fixed similarly: leotm/react-native-template-new-architecture#791

## Changelog

[iOS] [Changed] - Update Podfile to allow `PRODUCTION=1 pod install`

[CATEGORY] [TYPE] - Message

Pull Request resolved: #34234

Test Plan: Everything builds and runs as expected

Reviewed By: cortinico

Differential Revision: D38029117

Pulled By: cipolleschi

fbshipit-source-id: bdb58200a999cb66f1043a2feb670f9037c8e463
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cocoapods CocoaPods changes for building iOS and tvOS flipper Flipper changes ios Native iOS changes (Swift, C++, CocoaPods) p: Facebook Partner: Meta release mode Production build variant
Projects
None yet
Development

Successfully merging this pull request may close these issues.

iOS release needs Flipper disabled (fixed in 0.69.2/0.70)
1 participant