Hi Till,
Thanks! Perhaps a test on non-valid JSON would be nice?
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-error-code-fixes/+merge/126060
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
The proposal to merge lp:~zorba-coders/zorba/jsoniq-error-code-fixes into
lp:zorba has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-error-code-fixes/+merge/126060
--
https://code.launchpad.net/~zorba-coders/zorba
Thanks! Perhaps a test on non-valid JSON would be nice?
Yes, they're in now.
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-error-code-fixes/+merge/126060
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/jsoniq-error-code-fixes-2012-09-26T15-21-45.457Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-error-code-fixes/+merge/126060
Your team Zorba Coders is subscribed to branch
Review: Approve
Thanks!
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-error-code-fixes/+merge/126060
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe :
Validation queue job jsoniq-error-code-fixes-2012-09-26T15-21-45.457Z is
finished. The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-error-code-fixes/+merge/126060
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/jsoniq-error-code-fixes into
lp:zorba has been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-error-code-fixes/+merge/126060
--
https://code.launchpad.net/~zorba-coders/zorba
Hi Till,
JNDY0040 does not exist in the spec. The error for parse-json if the supplied
string is not valid JSON or if there is more than one object/array when it's
forbidden, is JNDY0021.
Does it make sense?
--
The proposal to merge lp:~zorba-coders/zorba/jsoniq-error-code-fixes into
lp:zorba has been updated.
Commit Message changed to:
s/JNTY0003/JNTY0024/
s/JSDY0020/JNTY0020/
s/JSDY0021/JNDY0021/
removed JSDY0040
doc fixes for jn:parse-json
For more details, see:
https://code.launchpad.net/~zorba
The proposal to merge lp:~zorba-coders/zorba/jsoniq-error-code-fixes into
lp:zorba has been updated.
Description changed to:
s/JNTY0003/JNTY0024/
s/JSDY0020/JNTY0020/
s/JSDY0021/JNDY0021/
removed JSDY0040
doc fixes for jn:parse-json
For more details, see:
https://code.launchpad.net/~zorba
The proposal to merge lp:~zorba-coders/zorba/jsoniq-error-code-fixes into
lp:zorba has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-error-code-fixes/+merge/126060
--
https://code.launchpad.net/~zorba-coders/zorba
JNDY0040 does not exist in the spec. The error for parse-json if the supplied
string is not valid JSON or if there is more than one object/array when it's
forbidden, is JNDY0021.
Does it make sense?
Yes, it does. I've removed JNDY0040 completely and replaced references to it
with JNDY0021.
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/jsoniq-error-code-fixes-2012-09-25T17-45-49.152Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-error-code-fixes/+merge/126060
Your team Zorba Coders is subscribed to branch
Validation queue job jsoniq-error-code-fixes-2012-09-25T17-45-49.152Z is
finished. The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-error-code-fixes/+merge/126060
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1,
Needs Fixing 1, Pending 1. Got: 1 Approve, 1 Pending.
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-error-code-fixes/+merge/126060
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
The proposal to merge lp:~zorba-coders/zorba/jsoniq-error-code-fixes into
lp:zorba has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-error-code-fixes/+merge/126060
--
https://code.launchpad.net/~zorba-coders/zorba
Till Westmann has proposed merging
lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba.
Commit message:
s/JNTY0003/JNTY0024/
s/JSDY0040/JNDY0040/
s/JSDY0020/JNTY0020/
s/JSDY0021/JNDY0021/
doc fixes for jn:parse-json
Requested reviews:
Ghislain Fourny (gislenius)
Till Westmann
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-error-code-fixes/+merge/126060
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe :
The proposal to merge lp:~zorba-coders/zorba/jsoniq-error-code-fixes into
lp:zorba has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-error-code-fixes/+merge/126060
--
https://code.launchpad.net/~zorba-coders/zorba
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/jsoniq-error-code-fixes-2012-09-24T17-47-49.386Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-error-code-fixes/+merge/126060
Your team Zorba Coders is subscribed to branch
Validation queue job jsoniq-error-code-fixes-2012-09-24T17-47-49.386Z is
finished. The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-error-code-fixes/+merge/126060
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1,
Needs Fixing 1, Pending 1. Got: 1 Approve, 1 Pending.
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-error-code-fixes/+merge/126060
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
The proposal to merge lp:~zorba-coders/zorba/jsoniq-error-code-fixes into
lp:zorba has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-error-code-fixes/+merge/126060
--
https://code.launchpad.net/~zorba-coders/zorba
23 matches
Mail list logo