-
-
Notifications
You must be signed in to change notification settings - Fork 407
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
Support destructuring assignment #265
Comments
I'd like to work on this, if that's good? |
Go for it @vapour101 ! |
Sorry, seems like I had less time to devote to this than I realised. I'm not going to be able to complete this issue. |
@afinch7 Could I have a try? |
@SamuelQZQ Are you still working on this? I just started some basic implementation and am currently at +128 tests passing. I think I could pick this up if you are not working on it. @davimiku you referenced #1324, that seems to implement shorthand syntax for Object Initializers right? I think this issue would be also about Destructuring Binding Patterns. Just to make sure I'm not workin on something you already started. |
@raskad That's correct, #1324 only implements initializing objects with the short-hand notation (on the "right-hand side"), i.e. const message = "hello"
const obj = { message } That PR didn't cover destructuring assignment (on the "left-hand side"), so great that you've started! I didn't start / wasn't planning on implementing destructuring assignment. Maybe if you don't hear from SamuelQZQ try asking a maintainer to change the assignment? |
@raskad Go ahead. I gave up. Haha. |
Support destructuring assignment syntax:
The text was updated successfully, but these errors were encountered: