ianmcook commented on PR #500: URL: https://github.com/apache/arrow-js/pull/500#issuecomment-6044331445
Looks good to me! A couple of nits found with help from Claude Opus 5.5: - The `as any` casts in the test aren't needed. `Other` has type `typeof Arrow`, and tsc passes with the casts removed. - This is the first test in the repo to import `@jest/globals`. That package isn't a declared devDependency and only resolves because jest depends on 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
