Skip to content

Commit

Permalink
build: use explicit yarn script to run lerna prepare
Browse files Browse the repository at this point in the history
this may deserve a refactor, but for now it hooks the v6 lerna prepare
to our yarn run so tests:jest passes and packages are ready
  • Loading branch information
mikehardy committed Oct 26, 2023
1 parent 8e3f5c5 commit c42688c
Show file tree
Hide file tree
Showing 7 changed files with 10 additions and 10 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/create_test_patches.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ jobs:
timeout_minutes: 15
retry_wait_seconds: 60
max_attempts: 3
command: DETOX_DISABLE_POSTINSTALL=1 yarn
command: DETOX_DISABLE_POSTINSTALL=1 yarn && yarn lerna:prepare

- name: Create Patches
run: |
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/linting.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
timeout_minutes: 15
retry_wait_seconds: 30
max_attempts: 3
command: yarn
command: yarn && yarn lerna:prepare
- name: Lint
uses: nick-fields/retry@v2
with:
Expand Down Expand Up @@ -83,7 +83,7 @@ jobs:
timeout_minutes: 15
retry_wait_seconds: 30
max_attempts: 3
command: yarn
command: yarn && yarn lerna:prepare
- name: Lint
run: yarn tsc:compile
- uses: actions/cache/save@v3
Expand Down Expand Up @@ -118,7 +118,7 @@ jobs:
timeout_minutes: 15
retry_wait_seconds: 30
max_attempts: 3
command: yarn
command: yarn && yarn lerna:prepare
- name: Yarn Install (Website)
uses: nick-fields/retry@v2
with:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ jobs:
timeout_minutes: 15
retry_wait_seconds: 60
max_attempts: 3
command: yarn
command: yarn && yarn lerna:prepare
- name: GIT Setup
run: |
git config --global user.name '@Salakar'
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/tests_e2e_android.yml
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ jobs:
timeout_minutes: 15
retry_wait_seconds: 60
max_attempts: 3
command: DETOX_DISABLE_POSTINSTALL=1 yarn
command: DETOX_DISABLE_POSTINSTALL=1 yarn && yarn lerna:prepare

- name: Cache Firestore Emulator
uses: actions/cache@v3
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/tests_e2e_ios.yml
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ jobs:
timeout_minutes: 15
retry_wait_seconds: 60
max_attempts: 3
command: yarn
command: yarn && yarn lerna:prepare

- name: Setup Ruby
uses: ruby/setup-ruby@v1
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/tests_jest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ jobs:
timeout_minutes: 15
retry_wait_seconds: 60
max_attempts: 3
command: yarn
command: yarn && yarn lerna:prepare
- name: Jest
run: yarn tests:jest-coverage
- uses: codecov/codecov-action@v3
Expand Down
4 changes: 2 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@
"private": true,
"scripts": {
"version": "node ./scripts/version.js",
"build": "lerna run prepare --parallel",
"lerna:prepare": "lerna run prepare --parallel",
"lerna:clean": "lerna clean",
"build:all:clean": "lerna run build:clean",
"build:all:build": "lerna run build",
"lint": "yarn lint:js && yarn lint:android && yarn lint:ios:check",
Expand All @@ -16,7 +17,6 @@
"lint:spellcheck": "spellchecker --quiet --files=\"docs/**/*.md\" --dictionaries=\"./.spellcheck.dict.txt\" --reports=\"spelling.json\" --plugins spell indefinite-article repeated-words syntax-mentions syntax-urls frontmatter",
"tsc:compile": "tsc --project .",
"lint:all": "yarn lint && yarn lint:markdown && yarn lint:spellcheck && yarn tsc:compile",
"lerna:clean": "lerna clean",
"tests:jest": "jest",
"tests:jest-watch": "jest --watch",
"tests:jest-coverage": "jest --coverage",
Expand Down

0 comments on commit c42688c

Please sign in to comment.