Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use @swc/jest instead of ts-jest where applicable #407

Closed
wants to merge 2 commits into from

Conversation

tjuranek
Copy link
Contributor

Overview

@swc/jest was recently found as a "faster drop in replacement for ts-jest". We see non-insignificant speed increases using this rust-based package over ts-jest.

This pull request updates applicable packages to use @swc/jest where it makes sense. Some notes:

  • @swc/jest doesn't work with stencil decorators. there is an open issue. Decorators on methods in object literals swc-project/swc#3062

  • the scene-composer package still relies on ts-jest for it's configuration, only the transformer was updated

  • source-iotsitewise and source-iottwinmaker tests weren't running before based on a clean install from the main branch, now they do. I'm unable to provide stats on any potential speed improvements.

  • any previously failing tests were left that way. there are a small number of these. these will be addressed at a later time.

  • I think we can get rid of a lot of the babel plugins by using @swc/jest. It'll require more investigation on a per-package basis to see what can and can't be removed.

Screen Shot 2022-11-30 at 4 35 05 PM

Legal

This project is available under the Apache 2.0 License.

@tjuranek tjuranek closed this Dec 6, 2022
@tjuranek tjuranek reopened this Dec 8, 2022
@tjuranek tjuranek self-assigned this Dec 8, 2022
@tjuranek tjuranek closed this Dec 13, 2022
@diehbria diehbria deleted the tsjest-to-swcjest branch March 16, 2023 00:32
fpauer added a commit that referenced this pull request Nov 15, 2024
Co-authored-by: Fernando Pauer <pauerf@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant