Skip to content

Commit 1234bf1

Browse files
kkafararthwood
andauthored
fix(CI): publish nightly - incorrect relative paths to scripts & missing build.log (#2891)
## Description Fixing issues making the CI not work! ## Changes * fixed relative paths to `set-version` script from `create-npm-package` script * removed job steps of `npm-screens-publish-nightly` workflow, since we do not generate `build.log` file at any point; we're also not including any `node_module` files into a release due to `whitelist` placed in `package.json`. ## Test code and steps to reproduce Let's see whether CI passes ## Checklist - [ ] Ensured that CI passes --------- Co-authored-by: Artur Bilski <[email protected]>
1 parent 56c6085 commit 1234bf1

File tree

2 files changed

+4
-14
lines changed

2 files changed

+4
-14
lines changed

.github/workflows/npm-screens-publish-nightly.yml

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -35,18 +35,6 @@ jobs:
3535
id: build
3636
run: ./scripts/create-npm-package.sh
3737

38-
- name: Check if any node_modules were packed
39-
id: node_modules
40-
run: ! grep --silent -E "node_modules/.+" build.log
41-
42-
- name: Show build log
43-
if: failure() && steps.build.outcome == 'failure'
44-
run: cat build.log
45-
46-
- name: Show packed node_modules
47-
if: failure() && steps.node_modules.outcome == 'failure'
48-
run: ! grep -E "node_modules/.+" build.log
49-
5038
- name: Add package name to env
5139
run: echo "PACKAGE_NAME=$(ls -l | egrep -o "react-native-screens-(.*)(=?\.tgz)")" >> $GITHUB_ENV
5240

scripts/create-npm-package.sh

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,17 @@
11
#!/bin/bash
22

3+
# This script assumes it is run from the top level repo directory.
4+
35
yarn install --immutable
46

5-
if ! CURRENT_VERSION=$(node ./set-version.js --nightly); then
7+
if ! CURRENT_VERSION=$(node ./scripts/set-version.js --nightly); then
68
exit 1
79
fi
810

911
yarn prepare
1012

1113
npm pack
1214

13-
node ./set-version.js "$CURRENT_VERSION" >/dev/null
15+
node ./scripts/set-version.js "$CURRENT_VERSION" >/dev/null
1416

1517
echo "Done!"

0 commit comments

Comments
 (0)