Skip to content

Conversation

@sh0ji
Copy link
Member

@sh0ji sh0ji commented Jan 21, 2026

This adds Codecov to Gamut, including an initial codecov.yml config, coverage reporting, and their new test analytics so we can try it out. Note that it only runs for PRs so we can test it. It should result in a report at https://app.codecov.io/gh/Codecademy/gamut/. If it works, I'll add it for the full repo.

I also made a few optimizations to the test workflow:

PR Checklist

  • Related to designs:
  • Related to JIRA ticket: GMT-1547
  • I have run this code to verify it works
  • This PR includes unit tests for the code change
  • This PR includes testing instructions tests for the code change
  • The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

@sh0ji sh0ji requested a review from a team as a code owner January 21, 2026 18:05
@sh0ji sh0ji requested review from LinKCoding and dreamwasp January 21, 2026 18:05
@sh0ji sh0ji changed the title Add CodeCov CI: Add CodeCov Jan 21, 2026
@sh0ji sh0ji changed the title CI: Add CodeCov ci: Add CodeCov Jan 21, 2026
@nx-cloud
Copy link

nx-cloud bot commented Jan 21, 2026

View your CI Pipeline Execution ↗ for commit 68858a2


☁️ Nx Cloud last updated this comment at 2026-01-23 17:29:15 UTC

@sh0ji sh0ji marked this pull request as draft January 21, 2026 18:12
sh0ji added 6 commits January 21, 2026 13:45
- we're just using the default coverageReporters: https://jestjs.io/docs/configuration#coveragereporters-arraystring--string-options
- the base config is using the preset, so we only need to declare reporters in one place
- 'summary' is an invalid reporter
- we want to always include the github-actions reporter in CI
these are different from the reporters
@codecov
Copy link

codecov bot commented Jan 21, 2026

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.

Thanks for integrating Codecov - We've got you covered ☂️

@sh0ji sh0ji changed the title ci: Add CodeCov ci: add Codecov Jan 21, 2026
sh0ji added 6 commits January 23, 2026 11:09
- nx run-many instead of nx affected to guarantee main tests always reflect the whole repo
- differentiate between PRs and main coverage with tags
use latest ubuntu LTS version
from https://docs.github.com/en/actions/reference/workflows-and-actions/events-that-trigger-workflows#pull_request:
> By default, a workflow only runs when a pull_request event's activity type is opened, synchronize, or reopened.
@github-actions
Copy link
Contributor

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