Skip to content

Commit 52290d1

Browse files
authored
Merge pull request #115 from weaviate/fix_ci_for_externals
Fix CI for PRs from externals
2 parents a1e7dca + 8139968 commit 52290d1

File tree

1 file changed

+14
-4
lines changed

1 file changed

+14
-4
lines changed

.github/workflows/tests.yaml

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,16 +16,26 @@ jobs:
1616
- uses: actions/setup-node@v3
1717
with:
1818
node-version: '16.x'
19-
- name: "Run tests"
19+
- name: "Install dependencies"
20+
run: |
21+
npm install
22+
ci/run_dependencies.sh
23+
- name: "Run tests with authentication tests"
24+
if: ${{ !github.event.pull_request.head.repo.fork }}
2025
env:
2126
OKTA_DUMMY_CI_PW: ${{ secrets.OKTA_DUMMY_CI_PW }}
2227
WCS_DUMMY_CI_PW: ${{ secrets.WCS_DUMMY_CI_PW }}
2328
run: |
24-
npm install
25-
ci/run_dependencies.sh
2629
npm test
27-
ci/stop_dependencies.sh
2830
npm run build
31+
- name: "Run tests without authentication tests (for forks)"
32+
if: ${{ github.event.pull_request.head.repo.fork }}
33+
run: |
34+
npm test
35+
npm run build
36+
- name: "Stop weavaite"
37+
run: ci/stop_dependencies.sh
38+
2939
deploy:
3040
needs: Tests
3141
if: github.event_name == 'push' && startsWith(github.ref, 'refs/tags')

0 commit comments

Comments
 (0)