Works with import about atom-node-resolver2

[Kikobeats]

Yes currently ES6 doesn't work.

Related: https://twitter.com/Kikobeats/status/777872609530740736

Definitely I need charliedowler's help for resolve this 😄 .

[charliedowler]

Hi,

I switched back to esprima and tried sourceType as you suggested. Imports won't work until this function is rewritten

astw(ast)(function(node) {
      if (!isRequire(node)) return

It's looking for require statements instead of both require and import, I don't think upgrading node-resolver to fully support es6 is as simple as bumping the esprima dependency.

What do you think?

Cheers,
Charlie

[Kikobeats]

Actually I don't know how to walk into the AST. If you feel confortable doing that I can add you as collaborator 😁

[Kikobeats]

Hey,

I released a new patch version. I switch the package used for parsing the AST. Looks like we are near to get this working!

[Kikobeats]

The current limitation is how to detect import.

This libraries uses https://github.com/hughsk/is-require/blob/master/index.js that very basic and it depends of the node that you pass.

I isolate the case in a little example: https://gist.github.com/Kikobeats/89168d73f57fd8e6c17c0effe1403987

Related: acornjs/acorn#441