[PATCH v6 1/8] models: Convert functions to properties

Stephen Finucane stephen at that.guru
Tue Oct 18 20:19:24 AEDT 2016


On 2016-10-17 22:56, Daniel Axtens wrote:
> Stephen Finucane <stephen at that.guru> writes:
> 
>> -    def sync_person(self):
>> -        pass
>> -
> 
>> +    def sync_person(self):
>> +        pass
>> +
> 
> Not sure why you're moving sync_person here. Doesn't really matter or
> need a respin but it might be worth trying to drop these hunks when
> merging.

Ah - that was done just to group all '@property' items together. I'll 
actually drop it when applying seeing as it's useless anyway and removed 
in another patch.

Stephen


More information about the Patchwork mailing list