Page MenuHomeElementl

Don't collect coverage on every Jest run
ClosedPublic

Authored by dish on Oct 5 2020, 3:24 PM.

Details

Summary

We don't need to collect coverage on every yarn jest, and it gets in the way when running yarn jest --watch.

Just run it on the build.

Test Plan

yarn jest, yarn jest --collectCoverage

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dish requested review of this revision.Oct 5 2020, 3:42 PM

LGTM! Thanks for the efforts to make this and buildkite faster, especially for JS-only changes it's been awesome.

This revision is now accepted and ready to land.Oct 5 2020, 7:31 PM
This revision was automatically updated to reflect the committed changes.