Skip to content

Fix overriding EXCLUDED_ARCHS when installing Hermes on RN v0.71 - #39763

Merged
cipolleschi merged 1 commit into
react:0.71-stablefrom
ken0nek:ken0nek/fix-exclude-archs
Dec 19, 2023
Merged

Fix overriding EXCLUDED_ARCHS when installing Hermes on RN v0.71#39763
cipolleschi merged 1 commit into
react:0.71-stablefrom
ken0nek:ken0nek/fix-exclude-archs

Conversation

@ken0nek

Copy link
Copy Markdown
Contributor

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.stablehere

ref: #39060

Changelog:

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

Test Plan:

Tests pass

@facebook-github-botfacebook-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

@cipolleschicipolleschi left a comment

Copy link
Copy Markdown
Contributor

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
ken0nekforce-pushed the ken0nek/fix-exclude-archs branch from d668ad8 to c2ff291CompareOctober 4, 2023 19:10
@ken0nek

Copy link
Copy Markdown
ContributorAuthor

Resolved the conflict against 0.71-stable

@cipolleschi

Copy link
Copy Markdown
Contributor

/rebase

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA SignedThis 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

@ken0nek@cipolleschi@facebook-github-bot