Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

"await" is a reserved word in module context #151

Closed
jrvidal opened this issue Jun 19, 2015 · 3 comments
Closed

"await" is a reserved word in module context #151

jrvidal opened this issue Jun 19, 2015 · 3 comments
Labels
accepted bug

Comments

@jrvidal
Copy link
Contributor

jrvidal commented Jun 19, 2015

11.6.2.2 Future Reserved Words
@nzakas
Copy link
Member

nzakas commented Jun 19, 2015

I'm sorry, there's no enough information in this issue to do anything. Are you reporting a bug? If so, what code were you parsing and what was the expected output?

If not, please explain the purpose of this issue.

@jrvidal
Copy link
Contributor Author

jrvidal commented Jun 19, 2015

Sorry about my cryptic report!

Example:

espree.parse("var await = 1;", {ecmaFeatures: {modules: true}})

I would expect this to throw, since await is a "future reserved word" in module context. I think I can submit a PR for this, it's a pretty simple fix.

@nzakas
Copy link
Member

nzakas commented Jun 20, 2015

Gotcha! PR definitely welcome.

@nzakas nzakas closed this as completed in 95d388d Jun 26, 2015
nzakas added a commit that referenced this issue Jun 26, 2015
@nzakas
Merge pull request #152 from jrvidal/await-keyword-151
1b1967e 
Fix: "await" is a future reserved word (fixes #151)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
accepted bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@nzakas @jrvidal @gyandeeps