← Back to team overview

banking-addons-team team mailing list archive

Re: [Merge] lp:~endiansolutions/banking-addons/ab61-nl_rabo into lp:banking-addons

 

I don't have the context's knowledge, so just a nitpickings reviews.

LGTM, with minors nitpickings:

lines 346-372, l.413, it's preferred to use parenthesis instead of continuation \
Example:
    recparse["60F"] = ("line0"
                       "line1"
                       "line3")

l.387: parenthesis not needed around line
-- 
https://code.launchpad.net/~endiansolutions/banking-addons/ab61-nl_rabo/+merge/141149
Your team Banking Addons Team is subscribed to branch lp:banking-addons.


References