-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
F# pipeline operator #7898
F# pipeline operator #7898
Conversation
160072a
to
b41710e
Compare
I ended up implementing typechecking as well 😀 |
Babel PR babel/babel#9450 for reference |
Well, there's already was function body parsing in |
I don't think it's possible to implement all three proposals in Flow, because there isn't such thing as parser plugins. |
@mAAdhaTTah What you think? Is this still too early? |
Probably. I don't know the Flow team's policy around this sort of thing. I know TS won't implement anything until stage 3, especially after the pain they've had with decorators. I'll point out that the committee isn't necessarily sold on the operator at all and there is a significant contingent who prefer the bind operator. Not only are we risking implementing the wrong version of the operator, we're risking the syntax not advancing and needing to rip it out. |
Well, Flow supports nullish coalescing, and optional chaining under flag, but they're still stage 2, but doesn't support other proposals |
@mAAdhaTTah I thought bind operator was dropped in 2015 and always was stage 0? |
AFAIK, Bind is still Stage 1, but it's stalled out. However, we have gotten suggestions, including from committee members, about bind: tc39/proposal-pipeline-operator#110 and tc39/proposal-pipeline-operator#107 so I'm still viewing it as a competing proposal. |
670c52f
to
ee718a0
Compare
80e37d0
to
0b22afe
Compare
Closing since the PR has too many merge conflicts, and we do not plan to implement proposals before stage 3 |
Discussion: #5443
await
parsing.flowconfig
option