Skip to content

ci: add GitHub Actions workflow with coverage reporting - #3

Merged
notandrewyang merged 3 commits into
mainfrom
ci/add-sdk-workflow
Sep 1, 2026
Merged

notandrewyang merged 3 commits into
mainfrom
ci/add-sdk-workflow

Conversation

@lukeGao0

Copy link
Copy Markdown
Contributor

Summary

  • Adds .github/workflows/ci.yml: typecheck → build → test → coverage, uploads lcov to Codecov
  • Wires real build and coverage badges into README (replaces nothing — these are new)
  • Disables the unvalidated coverage thresholds in vitest.config.ts until calibrated against a real CI run

Test plan

  • CI run on this PR passes (typecheck/build/coverage)
  • Codecov receives the lcov report and coverage badge renders
  • Follow-up: recalibrate thresholds in vitest.config.ts against the real coverage number, then re-enable

Adds CI pipeline (typecheck, build, test, coverage upload to Codecov)
and wires build/coverage badges into the README. Coverage thresholds
in vitest.config.ts are disabled until calibrated against a real run.
@lukeGao0
lukeGao0 requested a review from a team August 31, 2026 10:01
Comment thread .github/workflows/ci.yml Fixed
Tokenless upload is no longer supported by Codecov, so the upload
step failed in CI (harmlessly, due to fail_ci_if_error: false). Keep
coverage running locally in the pipeline for visibility, but drop the
badge and upload step until a real token is set up.
Comment thread .github/workflows/ci.yml
Comment on lines +11 to +24
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: pnpm/action-setup@v4
with:
version: 8.14.1
- uses: actions/setup-node@v4
with:
node-version: '20'
cache: 'pnpm'
- run: pnpm install --frozen-lockfile
- run: pnpm run typecheck
- run: pnpm run build
- run: pnpm run coverage
Enable thresholds using PR #3's measured coverage (93.02% stmts,
90.65% branch, 96.62% funcs, 93.02% lines) minus a safety margin.
@lukeGao0

lukeGao0 commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

reviewed

1 similar comment
@notandrewyang

Copy link
Copy Markdown
Contributor

reviewed

@notandrewyang
notandrewyang self-requested a review September 1, 2026 10:12
@notandrewyang
notandrewyang merged commit 63923ff into main Sep 1, 2026
4 checks passed
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.

4 participants