Skip to content

Conversation

@yungsters
Copy link
Contributor

@yungsters yungsters commented Sep 28, 2022

Summary:
Switches the Windows Hermes CI step to use Choco-Install which implicitly supports retries for more reliable builds.

See: actions/runner-images#721

Changelog:
[Internal]

Differential Revision: D39890407

Summary:
Moves the `retry3` utility function into its own file so that it can be reused in other steps that are not related to Android.

Changelog:
[Internal]

Differential Revision: D39889996

fbshipit-source-id: a879b665a94c78827629b901d053760b1532ca29
Summary:
I am not totally sure whether this will work, so putting up a PR to see what happens.

See: actions/runner-images#721

Changelog:
[Internal]

Differential Revision: D39890407

fbshipit-source-id: 567ba9926d8fa696163d5e0a97e057da0e9c666d
@yungsters yungsters requested a review from hramos as a code owner September 28, 2022 16:21
@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. p: Facebook Partner: Facebook Partner fb-exported labels Sep 28, 2022
@facebook-github-bot
Copy link
Contributor

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

@yungsters yungsters changed the title RN: Try Choco-Install from Windows Hermes CI RN: Switch to Choco-Install from Windows Hermes CI Sep 28, 2022
@analysis-bot
Copy link

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 7,728,343 +0
android hermes armeabi-v7a 7,131,643 +0
android hermes x86 8,035,627 +0
android hermes x86_64 8,009,014 +0
android jsc arm64-v8a 9,596,968 +0
android jsc armeabi-v7a 8,363,291 +0
android jsc x86 9,541,198 +0
android jsc x86_64 10,133,593 +0

Base commit: 7f061f8
Branch: main

@analysis-bot
Copy link

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

Base commit: 7f061f8
Branch: main

@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @yungsters in 1a28058.

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 Sep 28, 2022
@facebook-github-bot
Copy link
Contributor

This pull request has been reverted by 8bb9717.

OlimpiaZurek pushed a commit to OlimpiaZurek/react-native that referenced this pull request May 22, 2023
Summary:
Pull Request resolved: facebook#34811

Switches the Windows Hermes CI step to use `Choco-Install` which implicitly supports retries for more reliable builds.

See: actions/runner-images#721

Changelog:
[Internal]

Reviewed By: rickhanlonii

Differential Revision: D39890407

fbshipit-source-id: 69c03845d2f2160fb39219dd4a5fff90a00de278
@yungsters yungsters deleted the export-D39890407 branch July 11, 2023 00:23
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. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner Reverted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants