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.

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

Static class fields error: 'super' keyword outside a method. #3702

Closed
ChocolateLoverRaj opened this issue Jul 29, 2020 · 7 comments · Fixed by #3720
Closed

Static class fields error: 'super' keyword outside a method. #3702

ChocolateLoverRaj opened this issue Jul 29, 2020 · 7 comments · Fixed by #3720

Comments

@ChocolateLoverRaj
Copy link

  • Rollup Version: 2.23.0
  • Operating System (or Browser): Windows
  • Node Version (if applicable):
  • Link to reproduction (IMPORTANT, read below): https://rollup.js.org/repl/

Expected Behavior

Should've have this error. There is no javascript error when running it.

Actual Behavior

Doesn't work and shows error.

@lukastaegert
Copy link
Member

Apparently this is an acorn issue again, I created an upstream issue: acornjs/acorn-static-class-features#7

@ChocolateLoverRaj
Copy link
Author

So is this issue just waiting for someone to respond to the acornjs issue?

@lukastaegert
Copy link
Member

Exactly

@ChocolateLoverRaj
Copy link
Author

How long do you think it will take for acornjs to look at the issue? It doesn't seem like it is very active.

@lukastaegert
Copy link
Member

I guess any outside support is welcome.

@ChocolateLoverRaj
Copy link
Author

The upstream issue has been closed.

I wonder why.
There wasn't even a closing comment saying anything.

@ChocolateLoverRaj
Copy link
Author

My Mistake

I didn't realize that the closed issue is linked to a commit. I think they fixed it in the upstream repo.

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

Successfully merging a pull request may close this issue.

2 participants