Skip to content

Generalize RN build triggers/guard for future trains + bump version to 260318099.0.0#2101

Closed
cipolleschi wants to merge 2 commits into
static_hfrom
static_h-generic-staging-guard
Closed

Generalize RN build triggers/guard for future trains + bump version to 260318099.0.0#2101
cipolleschi wants to merge 2 commits into
static_hfrom
static_h-generic-staging-guard

Conversation

@cipolleschi

@cipolleschi cipolleschi commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Summary

Makes static_h the source of truth for the RN build's branch handling, so branches cut for future release trains (e.g. 26xxxx.*, 27xxxx.*) inherit the correct behavior automatically — no per-branch edits to rn-build-hermes.yml. Also advances the package version to the next train.

This is the generic form of what #2100 did for 260318099.0.0-staging.

1. Triggers → globs

push and pull_request now match static_h, '*-stable', and '*-staging' instead of listing individual branches. Any future stable/staging branch runs the RN build (and is verified on its PRs) out of the box. 250829098.0.0-stable is now covered by '*-stable'.

2. Dry-run guard → generalized

set_release_type forces RELEASE_TYPE=dry-run for static_h and any branch whose name ends in -staging, regardless of event — including workflow_dispatch, which has no branch filter and could otherwise be used to cut a real release from a staging branch.

if [[ $REF == "refs/heads/static_h" || $REF == refs/heads/*-staging ]]; then
  # forced dry-run

3. Bump version

npm/hermes-compiler/package.json bumped from 250829098.0.0 to 260318099.0.0 (the next release train's version).

Behavior

Ref Runs RN build Release allowed?
static_h yes no (forced dry-run)
*-staging (any) yes no (forced dry-run, even via dispatch)
*-stable (any) yes yes, via workflow_dispatch with release
PRs targeting the above yes dry-run

Note: pushing to a *-stable branch is dry-run (falls through to the else); an actual release still only happens via a manual workflow_dispatch selecting release on a stable branch — unchanged from today.

Test Plan

  • Verified the guard forces dry-run for static_h + *-staging and leaves *-stable releasable.
  • Verified the trigger globs parse to [static_h, *-stable, *-staging] for both push and pull_request.
  • Verified package.json is valid and version is 260318099.0.0.
  • CI on this PR should trigger "RN Build Static Hermes" in dry-run.

Make static_h the source of truth so branches cut for future release trains
(e.g. 26xxxx/27xxxx) inherit the correct behavior without manual edits:

- Triggers: use globs ('*-stable', '*-staging') for push and pull_request so
  any future stable/staging branch runs the RN build automatically.
- Guard: force dry-run for static_h and any '*-staging' branch (even via
  workflow_dispatch). '*-stable' branches are unaffected and can still release.

'250829098.0.0-stable' is now covered by the '*-stable' glob.
@meta-cla meta-cla Bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Jul 7, 2026
Advance static_h to the next release train's version.
@cipolleschi cipolleschi changed the title Generalize RN build triggers and dry-run guard for future release trains Generalize RN build triggers/guard for future trains + bump version to 260318099.0.0 Jul 7, 2026
@meta-codesync

meta-codesync Bot commented Jul 7, 2026

Copy link
Copy Markdown

@cipolleschi has imported this pull request. If you are a Meta employee, you can view this in D110871622.

@cortinico cortinico 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.

Review automatically exported from Phabricator review in Meta.

@meta-codesync meta-codesync Bot closed this in b56d339 Jul 7, 2026
@meta-codesync meta-codesync Bot added the Merged label Jul 7, 2026
@meta-codesync

meta-codesync Bot commented Jul 7, 2026

Copy link
Copy Markdown

@cipolleschi merged this pull request in b56d339.

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

Labels

CLA Signed Do not delete this pull request or issue due to inactivity. Merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants