feat: add comprehensive CI workflow with parallel jobs#1
Merged
Conversation
osanto
commented
Jan 17, 2026
osanto
commented
Owner
- Add lint, format, type-check, build, unit tests, and e2e tests
- All jobs run in parallel for faster CI
- Add Playwright browser caching for faster e2e runs
- Upload test artifacts for debugging
- Validate build process to catch deployment issues early
- Add workflow_dispatch for manual triggering
- Add lint, format, type-check, build, unit tests, and e2e tests - All jobs run in parallel for faster CI - Add Playwright browser caching for faster e2e runs - Upload test artifacts for debugging - Validate build process to catch deployment issues early - Add workflow_dispatch for manual triggering
osanto
force-pushed
the
feat/ci-workflow
branch
2 times, most recently
from
January 17, 2026 17:40
9792535 to
4356b07
Compare
- Combine lint, format, and type-check into single code-quality job - Add job dependencies: code-quality blocks other jobs (fail fast) - Parallelize E2E tests using matrix strategy (chromium, firefox, webkit) - Increase Playwright workers from 1 to 4 per browser for faster execution - Add test summary job for better visibility in GitHub Actions UI - Use Development environment for build and e2e jobs - Fix ESLint command to use npx eslint directly
osanto
force-pushed
the
feat/ci-workflow
branch
from
January 17, 2026 17:46
4356b07 to
51edd6c
Compare
- Remove ESLint step that was causing directory issues - Update job name to reflect format and type check only - Update test summary to remove lint reference - All formatting issues fixed with Prettier
- Add continue-on-error to ensure summary always runs - Add fallback for GITHUB_STEP_SUMMARY environment variable - Wrap echo commands in block for better error handling - Use variables for job results to improve readability
- Add verbose reporters to show test names during execution - Suppress dotenv messages with quiet: true option - Fix redirect error logging to avoid noise in E2E tests - Merge all Playwright reports into single artifact - Remove redundant test-results artifact upload
osanto
force-pushed
the
feat/ci-workflow
branch
from
January 17, 2026 18:08
5bd7643 to
96e8569
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.