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
« Back to merge proposal
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 initialized correctly
s/initiaized/
s/corectly/
- 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