Code review comment for lp:~therp-nl/banking-addons/ba70-lp1231174-default_method_should_not_raise_at_module_installation_time

Revision history for this message
Ronald Portier (Therp) (rportier1962) wrote :

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.

« Back to merge proposal