What is going on with the tests..

/home/travis/build/babel/babel/packages/babel-generator/node_modules/babylon/node_modules/babel-runtime/helpers/typeof.js:19
var _typeof = typeof _symbol2.default === "function" && /istanbul ignore next/(0, _typeof4.default)(_iterator2.default) === "symbol" ? function (obj) {
^
TypeError: (0 , _typeof4.default) is not a function

I think one issue is that pirates is not really reverting, before being called twice. which we currently do. And it is replacing module._compile but never reverting it.

@hzoo: Thanks for opening this!

@danez: Hmm... I'll try to look into that. (It might be a day or two, I'm a bit busy at the moment, sorry.)

@hzoo: Hmm... I have no idea why we're getting a TypeError, but I've been thinking about this: I think that it would be better to just fix babel's built-in hook, as I haven't really had time to maintain pirates, and I'm thinking about depreciating it. Would you be open to a PR for that?

Ah understandable, thats is fine too.

Ok let us know - closing

@ariporad what are your thoughts on deprecating pirates? It seems it would still be useful (eg for stacking babel and ts-node)

@graingert: My primary thoughts are this:

1. I'm quite busy at the moment, and haven't really been able to dedicate a lot of time to it. (For example, I haven't been able to finish writing the tests.)
2. I'm not entirely sure that I would be the proper person to maintain it, as I don't actually have any projects that use it.
3. I'm more than happy to add someone else as a contributor if they're interested. If someone else also really needs it, I could try to make it a higher priority.

So, TL;DR: I don't really have a whole lot of time to allocate to it atm. If someone else needs it, I'm happy to try and find some time to work on it. I'm also happy to add someone as a contributor.

@hzoo Would you consider applying this update without pirates. I just noticed that babel reading from the filesystem is the primary cause of incompatibility, but it's actually a really easy and quick fix to make it work. Here's the update I did in ts-node (the original hook was based on babel-register, so should look like a familiar change - TypeStrong/ts-node@44c389e#diff-f41e9d04a45c83f3b6f6e630f10117feR288). Basically, just use module._compile and fallback to the previous loader to pass through the contents instead of reading directly from fs.