Skip to content

chore: migrate jest to swc#1518

Merged
Rajesh-Nagarajan-11 merged 6 commits into
layer5io:masterfrom
rishiraj38:chore/migrate-jest-swc-recreate
May 11, 2026
Merged

chore: migrate jest to swc#1518
Rajesh-Nagarajan-11 merged 6 commits into
layer5io:masterfrom
rishiraj38:chore/migrate-jest-swc-recreate

Conversation

@rishiraj38
Copy link
Copy Markdown
Member

@rishiraj38 rishiraj38 commented May 8, 2026

Notes for Reviewers

This PR related to #1239

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request migrates the project's testing transformer from ts-jest to @swc/jest to improve performance. The changes include adding a .swcrc configuration file, updating jest.config.js to use the SWC transformer, and adjusting dependencies in package.json. Feedback was provided regarding the Jest configuration, suggesting the removal of inline transformer options to prevent them from overriding the global settings defined in the new .swcrc file.

Comment thread jest.config.js Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Copy Markdown
Contributor

@banana-three-join banana-three-join left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 4 changed files in this pull request and generated no new comments.

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 merged commit facf99a into layer5io:master May 11, 2026
9 checks passed
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.

4 participants