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

Move Jest config to TypeScript file #1885

Merged
merged 1 commit into from
Nov 6, 2024
Merged

Conversation

victorlin
Copy link
Member

@victorlin victorlin commented Nov 6, 2024

Description of proposed changes

Reasons:

  1. This is officially recommended https://jestjs.io/docs/configuration
  2. Reduces clutter in package.json
  3. Allows type checking and IDE hints for config options.

Checklist

@victorlin victorlin self-assigned this Nov 6, 2024
@nextstrain-bot nextstrain-bot temporarily deployed to auspice-victorlin-jest--kwycgs November 6, 2024 20:11 Inactive
@victorlin victorlin force-pushed the victorlin/jest-config branch from ae5847e to 1a1c62f Compare November 6, 2024 20:17
@victorlin victorlin temporarily deployed to auspice-victorlin-jest--kwycgs November 6, 2024 20:17 Inactive
@victorlin victorlin force-pushed the victorlin/jest-config branch from 1a1c62f to f7a186a Compare November 6, 2024 20:26
@victorlin victorlin temporarily deployed to auspice-victorlin-jest--kwycgs November 6, 2024 20:26 Inactive
@victorlin victorlin marked this pull request as ready for review November 6, 2024 20:32
Copy link
Member

@jameshadfield jameshadfield left a comment

Choose a reason for hiding this comment

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

Haven't tested thoroughly and it's been a while since I delved into jest but this looks like the recommended direction to go in and CI looks good.

Reasons:

1. This is officially recommended
   <https://jestjs.io/docs/configuration>
2. Reduces clutter in package.json
3. Allows type checking and IDE hints for config options.

ts-node is required to read the TypeScript configuration file.
@victorlin victorlin force-pushed the victorlin/jest-config branch from fef67b7 to f9b8ce9 Compare November 6, 2024 21:35
@victorlin victorlin temporarily deployed to auspice-victorlin-jest--kwycgs November 6, 2024 21:35 Inactive
@victorlin victorlin merged commit f046672 into master Nov 6, 2024
25 checks passed
@victorlin victorlin deleted the victorlin/jest-config branch November 6, 2024 21:37
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.

3 participants