banking-addons-team team mailing list archive
-
banking-addons-team team
-
Mailing list archive
-
Message #01557
Re: [Merge] lp:~therp-nl/banking-addons/7.0-no_unnecessary_raise into lp:banking-addons
Review: Approve code review
with line 14 removed, we might run into other exceptions (like unable to connect to some lookup service etc), whereas the #29 and #86 catch only NotImplemented. I still think it's safe to do that.
--
https://code.launchpad.net/~therp-nl/banking-addons/7.0-no_unnecessary_raise/+merge/205648
Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons.
References