Re: [PATCH v4] api: Add comments to patch and cover endpoints

2018-05-02 Thread Daniel Axtens
Veronika Kabatova <vkaba...@redhat.com> writes: > - Original Message - >> From: "Daniel Axtens" <d...@axtens.net> >> To: "Veronika Kabatova" <vkaba...@redhat.com> >> Cc: patchwork@lists.ozlabs.org >> Sent: Wednesday, Ma

Re: [PATCH v4] api: Add comments to patch and cover endpoints

2018-05-02 Thread Veronika Kabatova
- Original Message - > From: "Daniel Axtens" <d...@axtens.net> > To: "Veronika Kabatova" <vkaba...@redhat.com> > Cc: patchwork@lists.ozlabs.org > Sent: Wednesday, May 2, 2018 6:22:34 PM > Subject: Re: [PATCH v4] api: Add comments to pat

Re: [PATCH v4] api: Add comments to patch and cover endpoints

2018-04-30 Thread Veronika Kabatova
- Original Message - > From: "Daniel Axtens" <d...@axtens.net> > To: vkaba...@redhat.com, patchwork@lists.ozlabs.org > Sent: Monday, April 30, 2018 6:55:11 PM > Subject: Re: [PATCH v4] api: Add comments to patch and cover endpoints > > Apologies

Re: [PATCH v4] api: Add comments to patch and cover endpoints

2018-04-30 Thread Daniel Axtens
Apologies if this is a silly question, but I was looking at this to check the perfomance, and I noticed that in the patch list endpoint (/api/patches/), it seems to be linking to the 'covers' comments endpoint rather than the 'patches' api endpoint: ... "delegate": null, "mbox":

Re: [PATCH v4] api: Add comments to patch and cover endpoints

2018-04-27 Thread Veronika Kabatova
- Original Message - > From: "Stephen Finucane" <step...@that.guru> > To: vkaba...@redhat.com, patchwork@lists.ozlabs.org > Sent: Friday, April 27, 2018 5:37:57 PM > Subject: Re: [PATCH v4] api: Add comments to patch and cover endpoints > > On Wed

Re: [PATCH v4] api: Add comments to patch and cover endpoints

2018-04-27 Thread Stephen Finucane
On Wed, 2018-04-25 at 19:33 +0200, vkaba...@redhat.com wrote: > From: Veronika Kabatova > > Signed-off-by: Veronika Kabatova Two comments below. I've actually addressed these already with my own patch. Assuming you're happy with said patch, I can