Veebers: in the future, please separate whitespace / PEP8 fixes from logic changes. It complicates the diff, makes diffs longer (which leads to fewer & poorer code reviews), and generally slows things down. I find it quite hard to see which changes in this diff I ought to be paying attention to, and which are harmless.
Veebers: in the future, please separate whitespace / PEP8 fixes from logic changes. It complicates the diff, makes diffs longer (which leads to fewer & poorer code reviews), and generally slows things down. I find it quite hard to see which changes in this diff I ought to be paying attention to, and which are harmless.