Stephen Finucane <step...@that.guru> writes: > On 2016-10-17 22:56, Daniel Axtens wrote: >> Stephen Finucane <step...@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.
Sorry to have taken so long to get back to you. Still working on the rest of the series. Given that you're dropping these: Reviewed-by: Daniel Axtens <d...@axtens.net> Regards, Daniel > > Stephen
signature.asc
Description: PGP signature
_______________________________________________ Patchwork mailing list Patchwork@lists.ozlabs.org https://lists.ozlabs.org/listinfo/patchwork