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

Revision history for this message
Matthias Brantner (matthias-brantner) 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

- the jsoniq functions module doesn't need to be included
- variable names and errors should not use camel-case but dashes, e.g. subRange => sub-range

review: Needs Fixing

« Back to merge proposal