[GitHub] [arrow] domoritz commented on a change in pull request #10673: ARROW-13277: [JS] Add declaration maps for TypeScript

[GitBox <gi...@apache.org>]

domoritz commented on a change in pull request #10673:
URL: https://github.com/apache/arrow/pull/10673#discussion_r665448616



##########
File path: js/jest.config.js
##########
@@ -45,7 +45,8 @@ module.exports = {
       "^.+\\.tsx?$": "ts-jest"
     },
     "transformIgnorePatterns": [
-      "/node_modules/(?!web-stream-tools).+\\.js$"
+      "/node_modules/(?!web-stream-tools).+\\.js$",
+      ".*/cjs/.*\\.js$"

Review comment:
       Without this, I am getting this strange error when I run `yarn gulp test:es5:cjs`?
   
   ```
   ts-jest[ts-compiler] (WARN) Unable to process '/Users/dominik/Code/arrow/js/targets/es5/cjs/io/adapters.js', falling back to original file content. You can also configure Jest config option `transformIgnorePatterns` to ignore /Users/dominik/Code/arrow/js/targets/es5/cjs/io/adapters.js from transformation or make sure that `outDir` in your tsconfig is neither `''` or `'.'`
   ```
   
   Any idea what's causing it?




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org