Code review comment for lp:~zorba-coders/zorba/f-nosql-db-trunk

Revision history for this message
Cezar Andrei (cezar-andrei) wrote :

> 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.

« Back to merge proposal