Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-26 Thread Ghislain Fourny
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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-26 Thread Till Westmann
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

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-26 Thread Till Westmann
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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-26 Thread Zorba Build Bot
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

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-26 Thread Ghislain Fourny
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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-26 Thread Zorba Build Bot
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:

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-26 Thread noreply
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

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-25 Thread Ghislain Fourny
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? --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-25 Thread Till Westmann
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-25 Thread Till Westmann
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-25 Thread Till Westmann
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

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-25 Thread Till Westmann
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.

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-25 Thread Zorba Build Bot
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-25 Thread Zorba Build Bot
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:

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-25 Thread Zorba Build Bot
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-25 Thread Zorba Build Bot
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-24 Thread Till Westmann
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

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-24 Thread 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-24 Thread Till Westmann
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-24 Thread Zorba Build Bot
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-24 Thread Zorba Build Bot
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:

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-24 Thread Zorba Build Bot
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-error-code-fixes into lp:zorba

2012-09-24 Thread Zorba Build Bot
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