> 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     : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

Reply via email to