jest icon indicating copy to clipboard operation
jest copied to clipboard

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

Open k-rajat19 opened this issue 1 year ago • 2 comments

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

k-rajat19 avatar Aug 05 '24 17:08 k-rajat19

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.

netlify[bot] avatar Aug 05 '24 17:08 netlify[bot]

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

peter-cardenas-ai avatar Aug 27 '24 18:08 peter-cardenas-ai

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.

github-actions[bot] avatar Nov 10 '24 00:11 github-actions[bot]