Code review comment for lp:~pedro.baeza/banking-addons/7.0-bank-statement-reconcile-prof_parsing
Revision history for this message
| Yannick Vaucher @ Camptocamp (yvaucher-c2c) wrote | # |
review:
Needs Fixing
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
73416ba
(Get the code!)
Next time someone review this module, he will wander why prof is passed to parser and why it is unused.
You should at least add a comment if you don't want it to be removed. Otherwise you should create a hook.