> I like the module a lot. There are only a couple of minor things that need > fixing: > > - There are a bunch of typos in the documentation. Could you please run it > through a spell-checker? > > For example, > > s/corespond/correspond > s/initiaized/initialized > s/corectly/correctly Fixed.
> > - the jsoniq functions module doesn't need to be included Yes it does, it's used at line 67: jn:members($helper-host-ports) > - variable names and errors should not use camel-case but dashes, e.g. > subRange => sub-range Done. -- https://code.launchpad.net/~zorba-coders/zorba/f-nosql-db-trunk/+merge/136215 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : [email protected] Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
