[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1207668 into lp:zorba

2013-08-02 Thread Federico Cavalieri
Federico Cavalieri has proposed merging lp:~zorba-coders/zorba/bug1207668 into lp:zorba. Commit message: Fixed bug 1207668 Requested reviews: Matthias Brantner (matthias-brantner) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1207668/+merge/178223 Fixed bug

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/feature-cloudant into lp:zorba

2013-08-02 Thread Federico Cavalieri
- The indentation has several problems. Could you please make it consistent. Especially in the C++ code. sure. - Why don't you use the same approach as we did for xbrl-cloud and keep the connection map in JSONiq (using the map module)? I think that would be more appropriate for this module

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/feature-external-cloudant-module into lp:zorba

2013-08-02 Thread Federico Cavalieri
Module in lp:~zorba-coders/zorba/cloudant-module -- https://code.launchpad.net/~zorba-coders/zorba/feature-external-cloudant-module/+merge/178255 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/feature-cloudant into lp:zorba

2013-08-02 Thread Federico Cavalieri
Review: Resubmit External module merge proposal in lp:~zorba-coders/zorba/feature-external-cloudant-module and lp:~zorba-coders/zorba/cloudant-module -- https://code.launchpad.net/~zorba-coders/zorba/feature-cloudant/+merge/177111 Your team Zorba Coders is subscribed to branch lp:zorba. --

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2013-08-02 Thread Paul J. Lucas
Paul J. Lucas has proposed merging lp:~paul-lucas/zorba/pjl-misc into lp:zorba. Commit message: Might as well change the public HexBinary API to match the new Base64 public API. Requested reviews: Paul J. Lucas (paul-lucas) For more details, see:

Re: [Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2013-08-02 Thread Paul J. Lucas
Review: Approve -- https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/178319 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 : https://launchpad.net/~zorba-coders

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2013-08-02 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/178319 -- https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/178319 Your team

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2013-08-02 Thread Zorba Build Bot
Validation queue starting for the following merge proposals: https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/178319 Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue -- https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/178319 Your team Zorba Coders is

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2013-08-02 Thread Zorba Build Bot
Voting criteria failed for the following merge proposals: https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/178319 : Votes: {'Approve': 1} -- https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/178319 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2013-08-02 Thread Zorba Build Bot
Validation queue result for https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/178319 Stage CommitZorba failed. Check console output at http://jenkins.lambda.nu/job/CommitZorba/81/console to view the results. -- https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/178319

Re: [Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2013-08-02 Thread Matthias Brantner
Review: Approve -- https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/178319 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 : https://launchpad.net/~zorba-coders

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/use-dataguide into lp:zorba

2013-08-02 Thread Paul J. Lucas
Review: Needs Fixing Please match the rest of the code style, e.g.: s/NULL/nullptr/ s/dataguide/dataguide_/ -- https://code.launchpad.net/~zorba-coders/zorba/use-dataguide/+merge/176385 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2013-08-02 Thread Zorba Build Bot
Validation queue starting for the following merge proposals: https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/178319 Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue -- https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/178319 Your team Zorba Coders is

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2013-08-02 Thread Zorba Build Bot
Validation queue succeeded - proposal merged! -- https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/178319 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:~paul-lucas/zorba/pjl-misc into lp:zorba

2013-08-02 Thread noreply
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/178319 -- https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/178319 Your team Zorba

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1188058 into lp:zorba

2013-08-02 Thread Luis Rodriguez Gonzalez
Luis Rodriguez Gonzalez has proposed merging lp:~zorba-coders/zorba/bug-1188058 into lp:zorba. Requested reviews: Luis Rodriguez Gonzalez (kuraru) Chris Hillery (ceejatec) Related bugs: Bug #1188058 in Zorba: Update non-core module system https://bugs.launchpad.net/zorba/+bug/1188058