fix: support MemberExpression.optional

[6utt3rfly]

Add support for MemberExpression.optional (but leave support for non-standard OptionalMemberExpression type)

see estree/estree#204

[jonschlinkert]

thanks, this is great!

[6utt3rfly]

Thank you! And it's a very well done project. I wasn't sure about the expressionAsync changes, but I just copied from the OptionalMemberExpression method which seemed safe enough 🤷🏻‍♀️.

I was testing with the jsep package and found only a few discrepancies (like BigInt support), but I like this project 😊

[jonschlinkert]

it's a very well done project.

thanks! I tried to put a lot of thought into it, I'm glad you like it, thanks!

like BigInt support

I thought we had BigInt support here. or do you mean on jsep? Anything we need to fix?

thanks again!

[6utt3rfly]

BigInt support here. or do you mean on jsep? Anything we need to fix?

Sorry for being unclear. I meant that jsep doesn't yet have BigInt support

[jonschlinkert]

Ah got it, thank you! Your contributions are always welcome! Thx again!

…

Sent from my iPhone

On Sep 29, 2021, at 11:21 AM, Shelly ***@***.***> wrote:  BigInt support here. or do you mean on jsep? Anything we need to fix? Sorry for being unclear. I meant that jsep doesn't yet have BigInt support — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.