Skip to content

Conversation

@sh0ji
Copy link
Member

@sh0ji sh0ji commented Jan 28, 2026

This skips the NX cache on main tests to guarantee that coverage always exists for it. I also refactored the conditional logic because the semantics weren't quite right but that it'll have the same effect.

PR Checklist

  • Related to designs:
  • Related to JIRA ticket: WPLAT-3438
  • 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

@nx-cloud
Copy link

nx-cloud bot commented Jan 28, 2026

View your CI Pipeline Execution ↗ for commit 3020c55


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

@sh0ji sh0ji marked this pull request as ready for review January 29, 2026 15:28
@sh0ji sh0ji requested a review from a team as a code owner January 29, 2026 15:28
@sh0ji sh0ji requested review from LinKCoding and aresnik11 January 29, 2026 15:28
npx nx run-many --target=test --all --parallel=4 --ci --coverage --runInBand --skip-nx-cache
else
npx nx run-many --target=test --all --parallel=4 --ci --coverage --runInBand
npx nx affected --target=test --parallel=4 --ci --coverage --runInBand
Copy link
Member

Choose a reason for hiding this comment

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

we might want to add https://github.com/nrwl/nx-set-shas above in this workflow, or add it to the yarn shared action so it happens consistently. It just normalizes the shas used for affected in various GHA cases where they might get messed up.

@github-actions
Copy link
Contributor

@sh0ji sh0ji merged commit db9a43b into main Jan 29, 2026
21 checks passed
@sh0ji sh0ji deleted the codecov-main-fix branch January 29, 2026 16:26
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.

3 participants