zorba-coders team mailing list archive
-
zorba-coders team
-
Mailing list archive
-
Message #16768
Re: [Merge] lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba
Review: Needs Fixing
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
--
https://code.launchpad.net/~zorba-coders/zorba/f-nosql-db-trunk/+merge/136215
Your team Zorba Coders is subscribed to branch lp:zorba.
Follow ups
References