← Back to team overview

zorba-coders team mailing list archive

Re: [Merge] lp:~zorba-coders/zorba/update3.0_data-cleaning-module into lp:zorba/data-cleaning-module

 

Review: Needs Fixing

1. Need to put <p/> tags between doc paragraphs, including after leading short description. (Remove <br/> tags.)

2. The error in currency-convert() should be in the module's own namespace/prefix, with ALL-CAPS name.

3. Based on the test cases, some other functions also raise errors, but those are not documented.
-- 
https://code.launchpad.net/~zorba-coders/zorba/update3.0_data-cleaning-module/+merge/170213
Your team Zorba Coders is subscribed to branch lp:zorba/data-cleaning-module.


References