Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: simplify CI matrix and more comprehensive tests #640

Merged
merged 35 commits into from
Dec 19, 2024
Merged
Changes from 1 commit
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
1e3eedd
ci(work-in-progress): simplify CI matrix and more comprehensive tests
haoqunjiang Dec 18, 2024
7ef4f77
ci: invoke cypress/playwright commands in their own project directories
haoqunjiang Dec 18, 2024
788e34d
ci: use filter instead of relying on working-directory due to weird b…
haoqunjiang Dec 18, 2024
b7417c1
ci: is it because of the `cd` command messed up the working directory?
haoqunjiang Dec 18, 2024
93f3eaf
ci: move playground to an outside directory to see if it works
haoqunjiang Dec 18, 2024
8cf9707
ci: add package.json in playground
haoqunjiang Dec 18, 2024
30e5c0e
ci: how about skipping playground cache?
haoqunjiang Dec 18, 2024
32fc1d4
ci: let's focus on test-build until it passed
haoqunjiang Dec 18, 2024
4c0479c
ci: forgot to add playground path
haoqunjiang Dec 18, 2024
a2a7a3c
ci: recursive install?
haoqunjiang Dec 18, 2024
228b568
ci: let's see if it works without moving the directory
haoqunjiang Dec 18, 2024
7f3aa21
ci: must move playground; but does it work without empty package.json?
haoqunjiang Dec 18, 2024
53c4572
ci: is there a difference between pnpm -r i and pnpm recursive install?
haoqunjiang Dec 18, 2024
b10ddc7
ci: i feel it could be an pnpm issue, so change working-directory to …
haoqunjiang Dec 18, 2024
113f84f
ci: try work without pnpm-lock.yaml
haoqunjiang Dec 18, 2024
f1ddf34
chore: typo
haoqunjiang Dec 18, 2024
4a81afa
ci: try not moving playground again
haoqunjiang Dec 18, 2024
361adc0
ci: see what happens when we remove the cache
haoqunjiang Dec 18, 2024
ca961e0
ci: is it related to the submodule thing?
haoqunjiang Dec 18, 2024
76ba294
ci: use standalone pnpm installation
haoqunjiang Dec 18, 2024
92a3818
ci: got it! it's because not checking out submodules leading to missi…
haoqunjiang Dec 18, 2024
6433f0b
ci: use artifact to share output
haoqunjiang Dec 18, 2024
4adfadc
ci: copilot generated an outdated action version!!!
haoqunjiang Dec 18, 2024
fa11155
ci: always install deps in root directory to avoid tsconfig warnings
haoqunjiang Dec 18, 2024
e53aa87
ci: it's more like 'verify' than 'test'
haoqunjiang Dec 18, 2024
1ae6114
ci: skip typescript build in playground to speed up CI
haoqunjiang Dec 18, 2024
217386c
ci: use a single job to verify various scripts
haoqunjiang Dec 18, 2024
db07bd2
ci: update e2e job to latest implementations
haoqunjiang Dec 18, 2024
98a9a26
ci: run build before test:e2e
haoqunjiang Dec 18, 2024
c8b8b80
ci: work-concurrency 1 for e2e test in case of race condition
haoqunjiang Dec 18, 2024
3dc1e2d
ci: typo
haoqunjiang Dec 18, 2024
f1b6684
ci: add Cypress component testing for projects without Vitest
haoqunjiang Dec 19, 2024
04af4bb
ci: also test nightwatch
haoqunjiang Dec 19, 2024
bc0bf0d
Revert "ci: also test nightwatch"
haoqunjiang Dec 19, 2024
34f0f0b
ci: we can add lint & format check to the CI even the snapshots aren'…
haoqunjiang Dec 19, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
ci: work-concurrency 1 for e2e test in case of race condition
haoqunjiang committed Dec 18, 2024
commit c8b8b803dd780250ebf2bc1a365d0b6cee911322
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
@@ -166,7 +166,7 @@ jobs:

- name: Run e2e test script
working-directory: ./playground
run: pnpm --filter "*${{ matrix.e2e-framework }}*" test:e2e
run: pnpm --filter --workspace-concurrency 1 "*${{ matrix.e2e-framework }}*" test:e2e

# FIXME: `--with-tests` folders
# FIXME: Cypress component testing for projects without Vitest