-
-
Notifications
You must be signed in to change notification settings - Fork 195
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.
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
Should throw on async () => await 5 ** 6;
#472
Comments
Can you please provide details in this issue? A link to a PR isn’t enough for us to evaluate this. |
I updated the issue description. |
Thanks. This is actually an issue with Acorn, which Espree wraps. I’d suggest opening an issue there. We can pull in the updated version when it’s fixed. |
I think acorn already fixed it acornjs/acorn@848d14e |
Awesome, then we can look at incorporating it. |
@fisker it seems that the latest Acorn still doesn't throw an error on this particular example. Can you open an issue on https://github.com/acornjs/acorn repository? |
* Breaking: acorn to 8.1.0 changelog: https://github.com/acornjs/acorn/blob/master/acorn/CHANGELOG.md#breaking-changes fixes #470, #472 * Update tests/lib/parse.js Co-authored-by: Milos Djermanovic <milos.djermanovic@gmail.com> * fix: add ecmaversion>5 * fix: remove #472 test case Co-authored-by: Milos Djermanovic <milos.djermanovic@gmail.com>
@mdjermanovic as you requested, I have opened an issue and created a PR to fix this in Acorn. |
Somehow, I missed this. Thank you @robpalme! |
@robpalme thanks! |
This is now fixed in the Acorn 8.2.2 release. |
You are my hero @robpalme. Thank you. |
espree
parse this asawait (5 ** 6)
In chrome, this is a syntax error
Babel PR babel/babel#12441
The text was updated successfully, but these errors were encountered: