My IDE's doing it automatically. Is it a big deal?


On Thu, Aug 15, 2013 at 8:53 PM, Ned Batchelder <notifications@github.com>wrote:

> In xblock/test/test_core.py:
>
> > @@ -535,7 +538,7 @@ def __init__(self, model_data):
> >      # each time it's actually queried, so that the caching is
> >      # doing the work to maintain object identity.
> >      model_data = MagicMock(spec=ModelData)
> > -    model_data.get = lambda name, default=None: [name]
> > +    model_data.get = lambda name, default = None: [name]
>
> "default=None" is a formal parameter with a default value, it should have
> no spaces around the equals sign. Same for the handful of other changes in
> this file.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/XBlock/pull/67/files#r5805789>
> .
>