Skip to content

Commit cfa25e0

Browse files
kelsetfacebook-github-bot
authored andcommitted
fix(release scripts): backport fixes from 0.71 into main (facebook#35258)
Summary: This PR backport two fixes we did in 0.71 to unblock the release process: * the change in `publish-npm` is needed because of the introduction of .strict() from facebook@4f3ca8f * the removal of the other script (added originally here facebook@e4b5d3e) is because: 1) that step is not needed anymore (we don't publish/upload hermes artifacts to the GH release) 2) by the time this job gets run the release crew has already setup the GH release 3) the logic for the versioning was broken and even on the 0.71-rc pipeline it was tagging stuff as 1000.0.0 ## Changelog <!-- Help reviewers and the release process by writing your own changelog entry. For an example, see: https://reactnative.dev/contributing/changelogs-in-pull-requests --> [Internal] [Fixed] - Fix release scripts for "release" pipeline scenario Pull Request resolved: facebook#35258 Test Plan: The fact that 0.71-rc0 was released is the ✅ for this. Reviewed By: jacdebug Differential Revision: D41120888 Pulled By: cipolleschi fbshipit-source-id: 06d108f0659ad1db53c6324fe1d735f52c34a3c5
1 parent ea73a66 commit cfa25e0

File tree

3 files changed

+5
-160
lines changed

3 files changed

+5
-160
lines changed

.circleci/config.yml

Lines changed: 0 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -1492,43 +1492,6 @@ jobs:
14921492
-d "{\"event_type\": \"publish\", \"client_payload\": { \"version\": \"${CIRCLE_TAG:1}\" }}"
14931493
# END: Stable releases
14941494

1495-
# START: Stables and commitlies
1496-
- when:
1497-
condition:
1498-
or:
1499-
- equal: [ "release", << parameters.release_type >> ]
1500-
- equal: [ "dry-run", << parameters.release_type >> ]
1501-
steps:
1502-
- run:
1503-
name: Install dependencies
1504-
command: apt update && apt install -y jq jo
1505-
- run:
1506-
name: Create draft GitHub Release and upload Hermes binaries
1507-
command: |
1508-
RELEASE_VERSION=$(cat build/.version)
1509-
if [[ << parameters.release_type >> == "release" ]]; then
1510-
GIT_TAG=$CIRCLE_TAG
1511-
elif [[ << parameters.release_type >> == "dry-run" ]]; then
1512-
GIT_TAG=v1000.0.0
1513-
fi
1514-
1515-
ARTIFACTS=("")
1516-
for build_type in "Debug" "Release"; do
1517-
TARBALL_FILENAME=$(node ./scripts/hermes/get-tarball-name.js \
1518-
--buildType $build_type \
1519-
--releaseVersion $RELEASE_VERSION)
1520-
1521-
ARTIFACTS+=("$HERMES_WS_DIR/hermes-runtime-darwin/$TARBALL_FILENAME")
1522-
done
1523-
1524-
./scripts/circleci/create_github_release.sh \
1525-
<< parameters.release_type >> \
1526-
$GIT_TAG \
1527-
$RELEASE_VERSION \
1528-
$GITHUB_TOKEN \
1529-
"${ARTIFACTS[@]}"
1530-
# END: Stable and commitlies
1531-
15321495
# -------------------------
15331496
# JOBS: Nightly
15341497
# -------------------------

scripts/circleci/create_github_release.sh

Lines changed: 0 additions & 123 deletions
This file was deleted.

scripts/publish-npm.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,11 @@ const argv = yargs
6464
type: 'boolean',
6565
default: false,
6666
})
67+
.option('r', {
68+
alias: 'release', // useless but needed for CI
69+
type: 'boolean',
70+
default: false,
71+
})
6772
.strict().argv;
6873
const nightlyBuild = argv.nightly;
6974
const dryRunBuild = argv.dryRun;

0 commit comments

Comments
 (0)