Skip to content

infra(e2e): fix enabling new arch for RNTester iOS in the E2E script#34882

Closed
kelset wants to merge 2 commits intomainfrom
kelset/change-script-to-use-new-arch
Closed

infra(e2e): fix enabling new arch for RNTester iOS in the E2E script#34882
kelset wants to merge 2 commits intomainfrom
kelset/change-script-to-use-new-arch

Conversation

@kelset
Copy link
Contributor

@kelset kelset commented Oct 6, 2022

Summary

Quick follow up to #34513 to fix an issue that has been bothering the release crew for a while: the iOS new arch component not working! Turns out, we're silly billies 🤣

Changelog

[Internal] [Changed] - Add new arch flag to iOS pod install command in E2E script

Test Plan

Everything works correctly now:

Screenshot 2022-10-06 at 14 20 09

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Contributor A React Native contributor. p: Microsoft Partner: Microsoft Partner labels Oct 6, 2022
@facebook-github-bot
Copy link
Contributor

@cipolleschi has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@analysis-bot
Copy link

Platform Engine Arch Size (bytes) Diff
ios - universal n/a --

Base commit: c9e6a60
Branch: main

@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @kelset in 6552d47.

When will my fix make it into a release? | Upcoming Releases

@react-native-bot react-native-bot added the Merged This PR has been merged. label Oct 7, 2022
@kelset kelset deleted the kelset/change-script-to-use-new-arch branch October 7, 2022 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Contributor A React Native contributor. Merged This PR has been merged. p: Microsoft Partner: Microsoft Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants