Skip to content

Commit ca4aaff

Browse files
cipolleschimeta-codesync[bot]
authored andcommitted
Rename use-hermes-nightly workflow input to use-hermes-prebuilt (#56756)
Summary: Pull Request resolved: #56756 The use-hermes-nightly boolean input of prebuild-ios-core.yml was misleadingly named. It controls whether to use the latest prebuilt Hermes from npm's latest-v1 dist-tag (not a 'nightly' build). Rename: - prebuild-ios-core.yml input: use-hermes-nightly -> use-hermes-prebuilt - Update description of the input - Update comment to move the TODO note inline - Update callers in nightly.yml and test-all.yml ## Changelog: [Internal] - Reviewed By: cortinico Differential Revision: D104649628 fbshipit-source-id: 71c75b07caebfe5e7d03e80d47cefba883843c9a
1 parent ac48b56 commit ca4aaff

3 files changed

Lines changed: 8 additions & 8 deletions

File tree

.github/workflows/nightly.yml

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

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

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,8 @@ 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'
11+
use-hermes-prebuilt:
12+
description: 'Whether to resolve Hermes from the latest-v1 npm dist-tag (true) or from the version pinned in version.properties (false)'
1313
type: boolean
1414
required: false
1515
default: false
@@ -47,10 +47,10 @@ jobs:
4747
- name: Set Hermes version
4848
shell: bash
4949
run: |
50-
# Nightly RN builds resolve Hermes from npm's latest-v1 dist-tag (TODO:
51-
# rename to 'latest' once V1 is the only Hermes on npm). Stable builds
52-
# use the version pinned in version.properties.
53-
if [ "${{ inputs.use-hermes-nightly }}" == "true" ]; then
50+
# Non-stable RN builds resolve Hermes from npm's latest-v1 dist-tag.
51+
# TODO: rename 'latest-v1' to 'latest' once V1 is the only Hermes on npm.
52+
# Stable builds use the version pinned in version.properties.
53+
if [ "${{ inputs.use-hermes-prebuilt }}" == "true" ]; then
5454
HERMES_VERSION="latest-v1"
5555
else
5656
HERMES_VERSION=$(sed -n 's/^HERMES_VERSION_NAME=//p' packages/react-native/sdks/hermes-engine/version.properties)

.github/workflows/test-all.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ jobs:
100100
prebuild_react_native_core:
101101
uses: ./.github/workflows/prebuild-ios-core.yml
102102
with:
103-
use-hermes-nightly: ${{ !endsWith(github.ref_name, '-stable') }}
103+
use-hermes-prebuilt: ${{ !endsWith(github.ref_name, '-stable') }}
104104
secrets: inherit
105105
needs: [prebuild_apple_dependencies]
106106

0 commit comments

Comments
 (0)