Re: [PR] fix some typos

2022-09-14 Thread Willy Tarreau
On Wed, Sep 14, 2022 at 11:05:30PM +0200, Tim Düsterhus wrote: > On 9/14/22 22:43, Willy Tarreau wrote: > > Thanks, but quite honnestly, even this one is not a commit message. > > There really is not much to be explained for some obvious typo fixes in > comments (i.e. not even changing the

Re: [PR] fix some typos

2022-09-14 Thread Tim Düsterhus
Willy, [dropping the original PR author from Cc, because my reply is not useful to them] On 9/14/22 22:43, Willy Tarreau wrote: Thanks, but quite honnestly, even this one is not a commit message. There really is not much to be explained for some obvious typo fixes in comments (i.e. not

Re: [PR] fix some typos

2022-09-14 Thread Willy Tarreau
Hi Tim, On Wed, Sep 14, 2022 at 07:44:27PM +0200, Tim Düsterhus wrote: > The fixes look correct to me, but the commit message is horrible. I've > attached the patch with a proper commit message (dropping the Signed-off-by > and adding a Co-authored-by). > > Best regards > Tim Düsterhus > From

Re: [PR] fix some typos

2022-09-14 Thread Tim Düsterhus
Willy, On 8/29/22 09:23, PR Bot wrote: Author: cui fliter Number of patches: 1 This is an automated relay of the Github pull request: fix some typos Patch title(s): fix some typos Link: https://github.com/haproxy/haproxy/pull/1843 Edit locally: wget