Skip to content
This repository has been archived by the owner on Jan 25, 2022. It is now read-only.

proposal moves forward to stage 3 \o/ #107

Merged
merged 1 commit into from
Jul 25, 2019
Merged

Conversation

bcoe
Copy link
Contributor

@bcoe bcoe commented Jul 25, 2019

Amazing work authors @claudepache, @gisenberg, @dustinsavery, and
champions @DanielRosenwasser and @jridgewell 🎉.

Amazing work authors @claudepache, @the_gisenberg, @dustinsavery, and
champions @DanielRosenwasser and @jridgewell 🎉.
@DanielRosenwasser DanielRosenwasser merged commit 73cf985 into tc39:master Jul 25, 2019
@Mouvedia
Copy link

Mouvedia commented Jul 25, 2019

…after all the issues raised were put to rest by committing 3 lines in the README.
?.[] is inconsistent and a terrible addition to my language.

Not many TC39 proposals have 70 issues closed. That number alone should make you wonder…

Thanks for participating.

@Whobeu
Copy link

Whobeu commented Jul 25, 2019

I hope this can make it into some version of Node 12.x.x. I've used this feature extensively in C# and it would be nice to reduce a lot of null checks in my JS modules.

@GaryGSC
Copy link

GaryGSC commented Jul 25, 2019

@Whobeu Unfortunately, that isn't very likely. This proposal would likely need to be implemented in V8, which is used by Node.

According to Node's roadmap, they're planning on stopping at V8 version 7.6 for Node 12, which has already been released and doesn't implement this proposal. 🙁

Basically, Node 12 enters LTS before they'll be able to get to it.

@GaryGSC
Copy link

GaryGSC commented Oct 25, 2019

Hey, guess what? Node 12 is ahead of their projected schedule. There's now an in-progress PR to backport V8 7.8 to Node 12, which supports optional chaining behind the V8 flag --harmony-optional-chaining.

@Whobeu
Copy link

Whobeu commented Oct 25, 2019

I wonder if Nullish Coalescing made it in too. There was some work done in one of the 7.8 revs to support it. Great to see Optional Chaining.

@gisenberg
Copy link
Member

gisenberg commented Oct 25, 2019

@Whobeu It looks like nullish coalescing is included in that PR as well.

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

Successfully merging this pull request may close these issues.

6 participants