Skip to content

Conversation

@ken0nek
Copy link
Contributor

@ken0nek ken0nek commented Oct 2, 2023

Summary:

  • Fix overriding EXCLUDED_ARCHS when installing Hermes on RN v0.71
  • Extract the logic and the tests from main

Recommended to create a pull request against 0.71.stable here

ref: #39060

Changelog:

[IOS] [FIXED] - don't override EXCLUDED_ARCHS when installing Hermes

Test Plan:

Tests pass

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 2, 2023
@fortmarek fortmarek requested a review from cipolleschi October 4, 2023 07:02
Copy link
Contributor

@cipolleschi cipolleschi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. You can squash and merge!

@ken0nek ken0nek force-pushed the ken0nek/fix-exclude-archs branch from d668ad8 to c2ff291 Compare October 4, 2023 19:10
@ken0nek
Copy link
Contributor Author

ken0nek commented Oct 4, 2023

Resolved the conflict against 0.71-stable

@cipolleschi
Copy link
Contributor

/rebase

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants