Skip to content

ci: split main.yml into build + test workflows (run tests in parallel)#3150

Open
bk201- wants to merge 6 commits into
mainfrom
dev/split-ci-workflows
Open

ci: split main.yml into build + test workflows (run tests in parallel)#3150
bk201- wants to merge 6 commits into
mainfrom
dev/split-ci-workflows

Conversation

@bk201-

@bk201- bk201- commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Splits the monolithic main.yml into separate workflows so the slow tests run in parallel with the fast build/analyzer checks.

Changes

  • build.yml (workflow Build): 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 (workflow Tests): unit (vitest:coverage) + integration as steps in one job; runs in parallel with build.yml on its own runner. Includes the Upload Code Coverage step ported from main.yml.
  • e2e.yml / nosql-language-service.yml: unchanged behavior; only stale main.yml references in comments updated.
  • Removed main.yml.
  • Merged latest origin/main (nl2query unit tests + coverage setup).

Note

Branch protection required status checks must be updated: old Build is gone; new contexts are Build / Build and Tests / Tests (plus the existing e2e / nosql checks).

@bk201- bk201- requested a review from a team as a code owner June 12, 2026 10:56
@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

🔨 Build (Compile, Lint, Prettier, l10n, Package)

🔗 Source

🧱 Step Results

  • Validate Version: ✅ success
  • Localization: ✅ success
  • Lint: ✅ success
  • Prettier: ✅ success
  • Compile (tsc): ✅ success
  • Package: ✅ success
  • External Skills (info): ✅ success

📦 Package Information

📥 Artifacts (run)

✅ Build Status

Compile, analyzers and packaging passed. See sibling comments for unit/integration and E2E results.

@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

🔬 NoSQL language-service integration

Commit: 87bc156
Pull Request: #3150 ci: split main.yml into build + test workflows (run tests in parallel)

🧪 Result

  • NoSQL integration tests: ✅ success

📥 Artifacts (run)

@github-actions

Copy link
Copy Markdown
Contributor

🎭 E2E Tests (Playwright + VS Code)

Commit: ab3f0e8
Pull Request: #3150 ci: split main.yml into build + test workflows (run tests in parallel)

🧪 Result

  • E2E Tests: ✅ success

📥 Artifacts (run)

Tip: the HTML report artifact contains a self-contained Playwright report.
Download the zip, extract, and open index.html — or run
npx playwright show-report <extracted-dir> for the interactive view.

@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

🧪 Tests (Unit + Integration)

Commit: 87bc156
Pull Request: #3150 ci: split main.yml into build + test workflows (run tests in parallel)

🧪 Results

  • Unit Tests (vitest): ✅ success
  • Integration Tests (extension host): ✅ success

@github-code-quality

github-code-quality Bot commented Jun 12, 2026

Copy link
Copy Markdown

Code Coverage Overview

Languages: TypeScript

TypeScript / code-coverage/vitest

The overall coverage remains at 73%, unchanged from the branch.


Updated June 12, 2026 12:10 UTC
Code Coverage is in Public Preview. Learn more and provide us with your feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant