search

jestjs / jest

Delightful JavaScript Testing.
https://jestjs.io
MIT License
44.12k stars 6.44k forks source link

feat: Stop using `ts-node` as a default loader for parsing TS configs #15241

Open k-rajat19 opened 1 month ago

k-rajat19 commented 1 month ago

Summary

Breaking Change

Currently ts-node is set as a default loader for parsing TS configs, this PR changes that behavior because now there are other good alternatives like esbuild-register so it's better to stop doing anything by default and let users specify which module they wanna use (according to their use case) in a docblock comment for loading their ts configs.

Test plan

Fixed e2e tests

netlify[bot] commented 1 month ago

Deploy Preview for jestjs ready!

Built without sensitive environment variables

Name Link
Latest commit 730cbdfa45eef71fae3b568e666006943c7e30e4
Latest deploy log https://app.netlify.com/sites/jestjs/deploys/66b12abc578ab9000825d1a2
Deploy Preview https://deploy-preview-15241--jestjs.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

peter-cardenas-ai commented 1 month ago

would be great if we could remove the peer dependency on ts-node as well 👀