This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Thanks @ziebam ! I had a look at the issues/PRs related to vitest.

Looks like one test is failing:

 ❯ @commitlint/cli/src/cli.test.ts  (57 tests | 1 failed) 20862ms
   × should throw when called without [input]
     → Test timed out in 5000ms.
If this is a long-running test, pass a timeout value as the last argument or configure it globally with "testTimeout".

⎯⎯⎯⎯⎯⎯⎯ Failed Tests 1 ⎯⎯⎯⎯⎯⎯⎯

 FAIL  @commitlint/cli/src/cli.test.ts > should throw when called without [input]
Error: Test timed out in 5000ms.
If this is a long-running test, pass a timeout value as the last argument or configure it globally with "testTimeout".
⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯[1/1]⎯

 Test Files  1 failed | 83 passed (84)
      Tests  1 failed | 1055 passed (1056)
   Start at  08:47:41
   Duration  22.33s (transform 3.14s, setup 0ms, collect 7.98s, tests 29.32s, environment 21ms, prepare 7.67s)

Did all tests pass for you locally?

@escapedcat, thanks for taking a look! Should be fixed now. It seems that the stdin() function hung indefinitely as it waited for stdin and the stream was never closed. I assume that's because execa, even when provided with input = '' (as is the default when the test's cli is called without arguments), it "writes" that empty string to stdin anyway and explicitly closes the stream. tinyexec doesn't do that, so I got rid of the ifs checking for input.length > 0 in the places I've used.

Sorry for that! I initially assumed the test failed because my machine isn't powerful enough (since it timed out), but it turns out I was wrong. 😄

Thanks for fixing this!
Wondering why execa isn't being removed from the lock-file. Maybe I'm missing something?

Thanks for fixing this! Wondering why execa isn't being removed from the lock-file. Maybe I'm missing something?

I think there are still downstream dependencies that use it? Namely: @lerna/child-process ([email protected]), lint-staged ([email protected]) and vitest ([email protected]). vitest is migrating to tinyexec, but the change hasn't made it to the release yet (vitest-dev/vitest#6454), not sure about the other two.

Got it, yes, makes sense. Thanks!