[PATCH v2 10/13] REST: Add id field to all endpoints

Stephen Fincane stephen at that.guru
Fri Nov 25 07:05:54 AEDT 2016


On Tue, 2016-11-22 at 12:18 -0600, Andy Doan wrote:
> On 11/19/2016 10:51 AM, Stephen Finucane wrote:
> > 
> > Signed-off-by: Stephen Finucane <stephen at that.guru>
> > Cc: Andy Doan <andy.doan at linaro.org>
> > ---
> >  patchwork/api/check.py   | 2 +-
> >  patchwork/api/patch.py   | 6 +++---
> >  patchwork/api/person.py  | 2 +-
> >  patchwork/api/project.py | 2 +-
> >  patchwork/api/user.py    | 2 +-
> >  5 files changed, 7 insertions(+), 7 deletions(-)
> 
> I think its worth adding a couple of lines of test code for this. We 
> have pretty good coverage so far, and it would be a shame to lose it
> for 
> something this easy.

I've done that and then some in a new precursor patch. Let me know what
you think.

Stephen


More information about the Patchwork mailing list