Skip to content

Add tests in CI not to break Hermes-Xcode integration - #37616

Closed
cipolleschi wants to merge 1 commit into
react:mainfrom
cipolleschi:export-D46265656
Closed

Add tests in CI not to break Hermes-Xcode integration#37616
cipolleschi wants to merge 1 commit into
react:mainfrom
cipolleschi:export-D46265656

Conversation

@cipolleschi

Copy link
Copy Markdown
Contributor

Summary:
Add tests in CircleCI to avoid that changes in Hermes can break the Xcode integration.

Changelog:

[Internal] - Add CI tests to avoid to break the Hermes-Xcode integration

Differential Revision: D46265656

@facebook-github-botfacebook-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. p: Facebook Partner: Facebook Partner fb-exported labels May 29, 2023
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D46265656

1 similar comment
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D46265656

@analysis-bot

analysis-bot commented May 29, 2023

Copy link
Copy Markdown
PlatformEngineArchSize (bytes)Diff
androidhermesarm64-v8a8,744,784+0
androidhermesarmeabi-v7a8,056,413-1
androidhermesx869,235,928+0
androidhermesx86_649,086,825-1
androidjscarm64-v8a9,307,489+2
androidjscarmeabi-v7a8,496,712+1
androidjscx869,369,697+2
androidjscx86_649,624,715+2

Base commit: d3e3599
Branch: main

@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D46265656

@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D46265656

@cipolleschi
cipolleschiforce-pushed the export-D46265656 branch 2 times, most recently from fa236e7 to 48c52bdCompareMay 30, 2023 09:35
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D46265656

@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D46265656

1 similar comment
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D46265656

Summary:
Pull Request resolved: #37616
Add tests in CircleCI to avoid that changes in Hermes can break the Xcode integration.
## Changelog:
[Internal] - Add CI tests to avoid to break the Hermes-Xcode integration
Reviewed By: cortinico
Differential Revision: D46265656
fbshipit-source-id: 49a36986e144149f0f412dde6488daba37c49fb6
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D46265656

@facebook-github-botfacebook-github-bot added the Merged This PR has been merged. label May 30, 2023
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request has been merged in b0864fd.

Szymon20000 pushed a commit to Szymon20000/react-native that referenced this pull request Jun 14, 2023
Summary:
Pull Request resolved: react#37616
Add tests in CircleCI to avoid that changes in Hermes can break the Xcode integration.
## Changelog:
[Internal] - Add CI tests to avoid to break the Hermes-Xcode integration
Reviewed By: cortinico
Differential Revision: D46265656
fbshipit-source-id: 176f1a33fe6944a68fb14b62e5c626fe30d296cc
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.fb-exportedMergedThis PR has been merged.p: FacebookPartner: FacebookPartner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@cipolleschi@facebook-github-bot@analysis-bot