Skip to content

feat(ci): 馃懛 Add Codecov coverage reporting - #1

Merged
haskou merged 3 commits into
mainfrom
feat/codecov
Jul 15, 2026
Merged

feat(ci): 馃懛 Add Codecov coverage reporting#1
haskou merged 3 commits into
mainfrom
feat/codecov

Conversation

@haskou

@haskou haskou commented Jul 15, 2026

Copy link
Copy Markdown
Owner

Changes

  • Match the @haskou/value-objects CI pattern with Node 24.18.0, Yarn caching, and a combined lint and coverage step.
  • Upload coverage/lcov.info through codecov-action@v5 while retaining the metrics export and tarball checks.
  • Add the Codecov badge, link the published documentation, and document the release branches.
  • Ignore .npmrc so local configuration and tokens cannot enter Git.

Motivation

The repository generated coverage without sending it to Codecov. The README also omitted the release branch convention and linked the documentation through a repository path.

Impact

CI sends each coverage report to Codecov through the CODECOV_TOKEN secret. A Codecov outage will not block the pipeline.

Merging this feat/* branch triggers the corresponding minor release.

Validation

  • yarn format:check
  • yarn test:coverage: 39 suites, 81 tests, and 100% coverage
  • yarn test:package
  • yarn pages:build

@codecov

codecov Bot commented Jul 15, 2026

Copy link
Copy Markdown

Welcome to Codecov 馃帀

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

鈩癸笍 You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered 鈽傦笍

@haskou
haskou marked this pull request as ready for review July 15, 2026 17:21
@haskou
haskou merged commit b0ed1c1 into main Jul 15, 2026
3 checks passed
@haskou
haskou deleted the feat/codecov branch July 15, 2026 17:21
github-actions Bot added a commit that referenced this pull request Jul 15, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

馃挕 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ba2d427c6f

鈩癸笍 About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 馃憤.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/ci.yml
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node-version }}
node-version: 24.18.0

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Keep CI coverage for the supported Node floor

In .github/workflows/ci.yml's test job, pinning the only validation lane to Node 24.18.0 means changes can now merge without running any build, typecheck, or package-entrypoint check on Node 22, even though package.json still declares "node": ">=22" and the installation docs say the package supports Node.js 22 and newer. For regressions that only affect the minimum supported runtime, CI will stay green; keep a Node 22 lane (even just the previous build && test:typecheck path) or narrow the advertised support.

Useful? React with 馃憤聽/ 馃憥.

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