Skip to content

Commit 0d64594

Browse files
committed
chore: use turbo instead of yarn workspace for CI tests
Yarn v4 with node-modules linker does not propagate root node_modules/.bin to workspace packages. Replace `yarn workspace $PKG test` with `yarn turbo run test --filter=$PKG` which runs from root context and correctly resolves binaries like eslint and size-limit.
1 parent a47d8fa commit 0d64594

2 files changed

Lines changed: 5 additions & 5 deletions

File tree

.github/workflows/callable-native-unit-tests.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,12 +25,12 @@ jobs:
2525
env:
2626
TEST_PACKAGE: ${{ matrix.package.name }}
2727
run: |
28-
yarn workspace $TEST_PACKAGE prepare:ios
29-
yarn workspace $TEST_PACKAGE test:ios
28+
yarn turbo run prepare:ios --filter=$TEST_PACKAGE
29+
yarn turbo run test:ios --filter=$TEST_PACKAGE
3030
- name: Run Android Tests
3131
working-directory: ./amplify-js
3232
env:
3333
TEST_PACKAGE: ${{ matrix.package.name }}
3434
run: |
35-
yarn workspace $TEST_PACKAGE prepare:android
36-
yarn workspace $TEST_PACKAGE test:android
35+
yarn turbo run prepare:android --filter=$TEST_PACKAGE
36+
yarn turbo run test:android --filter=$TEST_PACKAGE

.github/workflows/callable-unit-tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ jobs:
2424
working-directory: ./amplify-js
2525
env:
2626
TEST_PACKAGE: ${{ matrix.package.name }}
27-
run: yarn workspace $TEST_PACKAGE test
27+
run: yarn turbo run test --filter=$TEST_PACKAGE
2828
# disable temporarily
2929
# - name: Check coverage report existence
3030
# id: check-report

0 commit comments

Comments
 (0)