From 33e43e103a6101b3fe9e08ac5016649f3b51ff68 Mon Sep 17 00:00:00 2001 From: Steven Date: Fri, 22 Sep 2023 13:14:57 -0400 Subject: [PATCH] [ci] use `--output-logs=new-only` to only list tests that changed (#10570) This PR effectively reverts the change in https://github.com/vercel/vercel/pull/9172 since it is safe to skip tests that are not part of the turbo cache miss results. Previously, we were unable to do this because all of those jobs/checks were marked as required on the github repo settings. But now we have a single ["Summary" job](https://github.com/vercel/vercel/blob/f6e863d4bbdeb755eb89f1c621bf38dddc7399e1/.github/workflows/test.yml#L111-L128) that is marked as required at the repo level so now we don't need to create jobs anymore just to see the turbo cache hit go green. - [15f90b0](https://github.com/vercel/vercel/pull/10570/commits/15f90b057da8c31ba860ca87ffad5750120007c1) correctly skipped all e2e tests since none of the packages changed - [2613963](https://github.com/vercel/vercel/pull/10570/commits/26139634fb7ba1f65aabe0cfdc76f4bdb32df1e0) correctly ran the `vercel` e2e tests since the cli package was changed - [8488b05](https://github.com/vercel/vercel/pull/10570/commits/8488b0543b6c261d2433ba5f3de4af8a9caabe99) correctly runs tests for all packages since build-utils was changed and all packages depend on it Also see [Turbo `-output-logs` docs](https://turbo.build/repo/docs/reference/command-line-reference/run#--output-logs) for more. --- .changeset/very-happy-tests.md | 2 ++ utils/chunk-tests.js | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) create mode 100644 .changeset/very-happy-tests.md diff --git a/.changeset/very-happy-tests.md b/.changeset/very-happy-tests.md new file mode 100644 index 000000000..a845151cc --- /dev/null +++ b/.changeset/very-happy-tests.md @@ -0,0 +1,2 @@ +--- +--- diff --git a/utils/chunk-tests.js b/utils/chunk-tests.js index f758df230..8e1c16a9a 100644 --- a/utils/chunk-tests.js +++ b/utils/chunk-tests.js @@ -47,7 +47,7 @@ async function getChunkedTests() { `run`, ...scripts, `--cache-dir=.turbo`, - '--output-logs=full', + '--output-logs=new-only', '--log-order=stream', '--', '--', // need two of these due to pnpm arg parsing