Review: Approve LGTM
On a totally minor side, I'd prefer "import re" and using "re.escape" the code; The reason being that there are several "escape" functions in the stdlib (for xml, cgi, shell command lines, re...) and it is easier for the reader when coming on the code to understand the context with a namespaced call. -- https://code.launchpad.net/~camptocamp/banking-addons/improve_lookup/+merge/154730 Your team Banking Addons Team is subscribed to branch lp:banking-addons/bank-statement-reconcile-70. -- Mailing list: https://launchpad.net/~banking-addons-team Post to : [email protected] Unsubscribe : https://launchpad.net/~banking-addons-team More help : https://help.launchpad.net/ListHelp
