tilgovi commented on issue #113:
URL: 
https://github.com/apache/incubator-annotator/issues/113#issuecomment-913791968


   Ah, I see. It shouldn't be a problem that `@mdef/finder` is ESM-only, since 
we only support versions of Node that have stable ESM support. However, it 
looks like the landscape of TypeScript -> ESM -> Mocha is very fresh right now.
   
   It's easy to stop transforming the module syntax by changing our Babel 
configuration, but Mocha still won't recognize the `.ts` files as ESM. The 
`@babel/register` package doesn't support ESM, because ESM loader support in 
Node.js is still experimental. There are packages like `babel-register-esm` and 
`ts-node` that try to use this, and we would be able to use this with the 
`loader` option in newer versions of Mocha, but this all seems like a bleeding 
edge minefield right now.
   
   Alternatively, we might compile our dependencies, or at least that one, 
which is a change we could make in our Babel config.


-- 
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: dev-unsubscr...@annotator.apache.org

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


Reply via email to