Skip to content

Commit 5e3a3ba

Browse files
authored
Use pinned Hermes version when version.properties is not 1000.0.0 (facebook#56607)
1 parent 0004708 commit 5e3a3ba

5 files changed

Lines changed: 10 additions & 15 deletions

File tree

.github/actions/test-ios-helloworld/action.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,10 @@ runs:
2626
- name: Set nightly Hermes versions
2727
shell: bash
2828
run: |
29-
node ./scripts/releases/use-hermes-nightly.js
29+
HERMES_VERSION=$(sed -n 's/^HERMES_VERSION_NAME=//p' packages/react-native/sdks/hermes-engine/version.properties)
30+
if [ "$HERMES_VERSION" == "1000.0.0" ]; then
31+
node ./scripts/releases/use-hermes-nightly.js
32+
fi
3033
- name: Run yarn install again, with the correct hermes version
3134
uses: ./.github/actions/yarn-install
3235
- name: Download ReactNativeDependencies

.github/actions/test-ios-rntester/action.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,10 @@ runs:
3535
- name: Set nightly Hermes versions
3636
shell: bash
3737
run: |
38-
node ./scripts/releases/use-hermes-nightly.js
38+
HERMES_VERSION=$(sed -n 's/^HERMES_VERSION_NAME=//p' packages/react-native/sdks/hermes-engine/version.properties)
39+
if [ "$HERMES_VERSION" == "1000.0.0" ]; then
40+
node ./scripts/releases/use-hermes-nightly.js
41+
fi
3942
- name: Run yarn install again, with the correct hermes version
4043
uses: ./.github/actions/yarn-install
4144
- name: Prepare IOS Tests

.github/workflows/nightly.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ jobs:
2929
prebuild_react_native_core:
3030
uses: ./.github/workflows/prebuild-ios-core.yml
3131
with:
32-
use-hermes-nightly: true
3332
version-type: nightly
3433
secrets: inherit
3534
needs: [prebuild_apple_dependencies]

.github/workflows/prebuild-ios-core.yml

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,6 @@ on:
88
type: string
99
required: false
1010
default: ''
11-
use-hermes-nightly:
12-
description: 'Whether to use the hermes nightly build or read the version from the versions.properties file'
13-
type: boolean
14-
required: false
15-
default: false
1611

1712
jobs:
1813
build-rn-slice:
@@ -47,12 +42,9 @@ jobs:
4742
- name: Set Hermes version
4843
shell: bash
4944
run: |
50-
if [ "${{ inputs.use-hermes-nightly }}" == "true" ]; then
51-
# We are not publishing nightly versions of Hermes V1 yet.
52-
# For now, we can use the latest version of Hermes V1 published on maven and npm.
45+
HERMES_VERSION=$(sed -n 's/^HERMES_V1_VERSION_NAME=//p' packages/react-native/sdks/hermes-engine/version.properties)
46+
if [ "$HERMES_VERSION" == "1000.0.0" ]; then
5347
HERMES_VERSION="latest-v1"
54-
else
55-
HERMES_VERSION=$(sed -n 's/^HERMES_V1_VERSION_NAME=//p' packages/react-native/sdks/hermes-engine/version.properties)
5648
fi
5749
echo "Using Hermes version: $HERMES_VERSION"
5850
echo "HERMES_VERSION=$HERMES_VERSION" >> $GITHUB_ENV

.github/workflows/test-all.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -96,8 +96,6 @@ jobs:
9696

9797
prebuild_react_native_core:
9898
uses: ./.github/workflows/prebuild-ios-core.yml
99-
with:
100-
use-hermes-nightly: ${{ !endsWith(github.ref_name, '-stable') }}
10199
secrets: inherit
102100
needs: [prebuild_apple_dependencies]
103101

0 commit comments

Comments
 (0)