Skip to content
This repository has been archived by the owner on May 19, 2018. It is now read-only.

Parsing yield + dynamicImport #378

Closed
hzoo opened this issue Feb 23, 2017 · 3 comments · Fixed by #383
Closed

Parsing yield + dynamicImport #378

hzoo opened this issue Feb 23, 2017 · 3 comments · Fixed by #383

Comments

@hzoo
Copy link
Member

hzoo commented Feb 23, 2017

Ref babel/babel-preset-env#168

I believe preset-env -> async-to-gen and it fails to parse

Seems like this doesn't parse: http://astexplorer.net/#/gist/d793e984f4ccfe0ca80c3760490c2fc2/62f92b3ed7628599ad98e884a7e3c1ee25344798

function* a() {
  yield import('http');
}

@kesne

@kesne
Copy link
Contributor

kesne commented Feb 23, 2017

This is also tracked in webpack, for the acorn plugin that's here: https://github.com/kesne/acorn-dynamic-import

Probably should knock both out at once, most people using import() in webpack use babel-eslint, so having them match would be 💯

@hzoo
Copy link
Member Author

hzoo commented Feb 23, 2017

Also made webpack/webpack#4308 btw

On that note, should probably figure out how to make babel-eslint simpler now that we have estree again

@kesne
Copy link
Contributor

kesne commented Feb 28, 2017

Webpack will be fixed with this: kesne/acorn-dynamic-import#5

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants