Skip to content

ci(app-shell): add packages/core coverage diff reporting with octocov - #453

Merged
IzumiSy merged 11 commits into
mainfrom
ci/packages-core-coverage-octocov
Aug 20, 2026
Merged

IzumiSy merged 11 commits into
mainfrom
ci/packages-core-coverage-octocov

Conversation

@IzumiSy

@IzumiSy IzumiSy commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Motivation

We want packages/core coverage to be measured per PR against the current main baseline, with the diff posted back to the PR and updated as new commits are pushed.

The baseline should live in GitHub Actions artifacts so merges to main refresh the reference data without introducing another storage system.

Design Decision

Use octocov with GitHub Actions artifacts

octocov already supports both posting/updating PR comments and storing/loading prior reports from GitHub Actions artifacts. Using that built-in flow keeps the implementation to one workflow and one .octocov.yml file instead of custom artifact lookup and comment management code.

Keep the scope limited to packages/core

The workflow only runs the @tailor-platform/app-shell coverage task and only triggers for packages/core or coverage-workflow changes. That keeps runtime and noise down while matching the current request.

Summary

  • add a packages/core coverage script that emits LCOV for octocov
  • add .octocov.yml to store main reports in artifacts and diff PR coverage against them
  • add a GitHub Actions workflow that runs on both main pushes and PRs, updating a single managed coverage comment

@github-actions

github-actions Bot commented Aug 20, 2026 •

Copy link
Copy Markdown
Contributor

Code Metrics Report

Coverage Test Execution Time
90.0% 1m57s

Code coverage of files in pull request scope (0.0%)

Files Coverage
packages/core/package.json 0.0%

Reported by octocov

@IzumiSy
IzumiSy marked this pull request as ready for review August 20, 2026 04:44
@IzumiSy
IzumiSy requested a review from a team as a code owner August 20, 2026 04:44
@IzumiSy IzumiSy self-assigned this Aug 20, 2026
@IzumiSy
IzumiSy force-pushed the ci/packages-core-coverage-octocov branch from 1a94ad9 to cc3c3c9 Compare August 20, 2026 07:06
@IzumiSy
IzumiSy force-pushed the ci/packages-core-coverage-octocov branch from a1096c8 to fdbb011 Compare August 20, 2026 07:09
@IzumiSy

IzumiSy commented Aug 20, 2026

Copy link
Copy Markdown
Contributor Author

Just rebased the branch and fixed lockfile after approve.

@IzumiSy
IzumiSy merged commit edd5c72 into main Aug 20, 2026
9 checks passed
@IzumiSy
IzumiSy deleted the ci/packages-core-coverage-octocov branch August 20, 2026 07:14
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.

2 participants