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

ignore this in field definition value #52

Merged
merged 3 commits into from
Aug 31, 2022
Merged

ignore this in field definition value #52

merged 3 commits into from
Aug 31, 2022

Conversation

LongTengDao
Copy link
Contributor

No description provided.

@LongTengDao LongTengDao marked this pull request as ready for review September 28, 2019 13:39
@LongTengDao
Copy link
Contributor Author

LongTengDao commented Sep 28, 2019 •
edited
Loading

It seems that field definition parsing is not supported standardly yet.

@LongTengDao LongTengDao reopened this May 28, 2021
@LongTengDao
Copy link
Contributor Author

class field definition is stage 4 now, which is supported in acorn 8.2.0 and acorn-walk 8.1.0, so I reopen this pr.

@LongTengDao
Copy link
Contributor Author

mark: the class static block also need to add after this done.

@rollingversions
Copy link

rollingversions bot commented Aug 31, 2022 •
edited
Loading

Change Log for acorn-globals (6.0.0 → 6.0.1)

Bug Fixes

  • Ignore this in class property definitions

Edit changelog

@ForbesLindesay ForbesLindesay merged commit badc24f into ForbesLindesay:master Aug 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@LongTengDao @ForbesLindesay