Skip to content

(fix): jest shouldn't transform json files in node_modules#491

Closed
agilgur5 wants to merge 1 commit into
jaredpalmer:masterfrom
agilgur5:fix-jest-transform-json
Closed

(fix): jest shouldn't transform json files in node_modules#491
agilgur5 wants to merge 1 commit into
jaredpalmer:masterfrom
agilgur5:fix-jest-transform-json

Conversation

@agilgur5

@agilgur5 agilgur5 commented Feb 5, 2020

Copy link
Copy Markdown
Collaborator
  • not sure why ts-jest was picking these up and giving warnings, but
    TSDX's custom transformIgnorePatterns is different from jest's
    default (everything in node_modules), so set it to ignore json too

Fixes #161

@jaredpalmer is there a reason TSDX has a custom transformIgnorePattern? It's been like that since one of the first commits so idk.
Jest's default is just /node_modules/, so not sure why TSDX's is slightly different. To allow for transforming TS files in node_modules?

- not sure why ts-jest was picking these up and giving warnings, but
  TSDX's custom transformIgnorePatterns is different from jest's
  default (everything in node_modules), so set it to ignore json too
@agilgur5

agilgur5 commented Feb 19, 2020

Copy link
Copy Markdown
Collaborator Author

#513 fixes the root cause of this, so closing as unnecessary.

@agilgur5 agilgur5 closed this Feb 19, 2020
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.

jest occasionally tries to transform json files -- needs better transform regex

2 participants