Closed zudov closed 7 years ago
Updates needed to make it compile, work and pass the tests with 0.11 branch of compiler.
Note: this doesn't involve the changes needed to support corefn AST outputted by 0.11, that would be the next step.
purescript-foreign have now removed all the json-related functionality for good, so the biggest change is the switch to purescript-argonaut.
purescript-foreign
purescript-argonaut
readString
readInt
readIdent
readExpr
decodeJson
do
Either String
lmap
Hi @zudov. Thanks for taking a stab at this.
I'm going to close this PR since I was able to achieve 0.11 support in #40 with fewer changes. It also doesn't introduce the argonaut dependency or change the API.
Updates needed to make it compile, work and pass the tests with 0.11 branch of compiler.
purescript-foreign
have now removed all the json-related functionality for good, so the biggest change is the switch topurescript-argonaut
.readString
,readInt
,readIdent
,readExpr
) polymorphicdecodeJson
is used in many places now.decodeJson
is binded in thedo
-notationEither String
lmap
is used to annotate error messages that come from parsers allowing to 'trace' a path at which the failure occured.