facebook/flow
+
( -22)
-
// ^^^^^ shit, StrictReservedWord! rollback
// ^^^^^ shit, StrictReservedWord! rollback
// ^^^^^ shit, StrictReservedWord! what a surprise! show it to the user
// ^^ shit! unexpected =>! error again
// ^^^^^ shit, StrictReservedWord! it's ok, keep going
I'm sorry for not updating tests 😦`make` and `yarn install` both still failing for me.
Splitting this diff is rather difficult, I'm afraid. Sorry!
Updates the `statement` and `toplevels` functions in `statement.ml` to produce unit ASTs. Sorry for the big diff.
In general the reasons attached to refined variables is in a sorry state. Should
I promised Sam that I'd add this comment and then forgot. Sorry
sorry, no.
1. When an LSP proxy gets a first Initialize request over its connection, it should first check whether any other LSP proxies are connected to the same singleton backend server. If they aren't, fine, it can proceed. If there are, then it should fail the Initialize request, and give back a ResponseError saying "Sorry another client is connected at the moment" with the retry flag set to true.
The JSX stuff seems pretty spotty in some areas and I'm making it worse. Sorry.
Sorry about this. Didn't realize I had introduced it.
Now that the new gorgeous docs are out, I feel really sorry for not having helped like I promised and I think the minimum I can do is to fill some of the remaining the gaps.
First off, sorry this diff is so large. It was difficult to remove `RestT` bit by bit at this point while making sure that `RestT` didn't show up in the wrong place.
sorry for the microdiffs, i'm switching branches a lot to compare before and after, and landing these debugging changes makes it easier.
so yeah...sorry for the big commit.
so I landed the thing, but I should have tried rerunning it... sorry
|