← Back to team overview

banking-addons-team team mailing list archive

Re: lp:~therp-nl/banking-addons/ba70-lp1231174-default_method_should_not_raise_at_module_installation_time into lp:banking-addons

 

Just to continue on my last comment, even though it is a little bit off topic.

The test context == None in my test script should of course be replaced by context is None [thanks flake8], so is instead of ==.

Doing this makes the two line version of the code faster in case an empty context is passed. Otherwise the two line version is still slower.
-- 
https://code.launchpad.net/~therp-nl/banking-addons/ba70-lp1231174-default_method_should_not_raise_at_module_installation_time/+merge/187677
Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons.


References