ci: split main.yml into build + test workflows (run tests in parallel)#3150
ci: split main.yml into build + test workflows (run tests in parallel)#3150bk201- wants to merge 6 commits into
Conversation
# Conflicts: # .github/workflows/main.yml
🔨 Build (Compile, Lint, Prettier, l10n, Package)🔗 Source
🧱 Step Results
📦 Package Information
📥 Artifacts (run)
✅ Build StatusCompile, analyzers and packaging passed. See sibling comments for unit/integration and E2E results. |
🔬 NoSQL language-service integrationCommit: 87bc156 🧪 Result
📥 Artifacts (run)
|
🎭 E2E Tests (Playwright + VS Code)Commit: ab3f0e8 🧪 Result
📥 Artifacts (run)
|
🧪 Tests (Unit + Integration)Commit: 87bc156 🧪 Results
|
Code Coverage OverviewLanguages: TypeScript TypeScript / code-coverage/vitestThe overall coverage remains at 73%, unchanged from the branch. Updated |
… concurrency, no-emoji job name
Splits the monolithic
main.ymlinto separate workflows so the slow tests run in parallel with the fast build/analyzer checks.Changes
build.yml(workflowBuild): single job with sequential steps — version validation, install, l10n check, verify-skills (warning-only), lint, prettier, compile (tsc), package + VSIX upload. Analyzer steps run even if one fails (you see every problem in one run), and any failure reds the job.test.yml(workflowTests): unit (vitest:coverage) + integration as steps in one job; runs in parallel withbuild.ymlon its own runner. Includes the Upload Code Coverage step ported frommain.yml.e2e.yml/nosql-language-service.yml: unchanged behavior; only stalemain.ymlreferences in comments updated.main.yml.origin/main(nl2query unit tests + coverage setup).Note
Branch protection required status checks must be updated: old
Buildis gone; new contexts areBuild / BuildandTests / Tests(plus the existing e2e / nosql checks).