[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/2] docs: Fix botched rST conversion of 'submitting-a-patch.
From: |
Peter Maydell |
Subject: |
Re: [PATCH 1/2] docs: Fix botched rST conversion of 'submitting-a-patch.rst' |
Date: |
Mon, 22 Nov 2021 13:53:19 +0000 |
On Mon, 22 Nov 2021 at 13:30, Thomas Huth <thuth@redhat.com> wrote:
>
> On 22/11/2021 14.25, Peter Maydell wrote:
> > On Mon, 22 Nov 2021 at 12:37, Thomas Huth <thuth@redhat.com> wrote:
> >> What about simply replacing it with a new sentence below the bullet list,
> >> saying:
> >>
> >> "Please also use a spell checker like `codespell
> >> https://github.com/codespell-project/codespell` with your patches"
> >
> > How many regular contributors actually do that?
>
> Considering the typos that we have in the code, not enough ;-)
>
> Anyway, it's just a polite recommendation here, not a must-do, so mentioning
> codespell here doesn't really hurt, does it?
I don't think we should be recommending to new contributors that
they do things that established contributors generally do not do.
The document has enough "things you should do or think about" already.
My preference would be for simply not mentioning spelling-checking.
(If we do want to come up with some process for dealing with
spelling issues in the codebase, then we either need to put it
into CI so it's run automatically, or we need to have something
that works on the individual patch level.)
-- PMM
- Re: [PATCH 1/2] docs: Fix botched rST conversion of 'submitting-a-patch.rst', (continued)
- Re: [PATCH 1/2] docs: Fix botched rST conversion of 'submitting-a-patch.rst', Thomas Huth, 2021/11/22
- Re: [PATCH 1/2] docs: Fix botched rST conversion of 'submitting-a-patch.rst', Kashyap Chamarthy, 2021/11/22
- Re: [PATCH 1/2] docs: Fix botched rST conversion of 'submitting-a-patch.rst', Thomas Huth, 2021/11/22
- Re: [PATCH 1/2] docs: Fix botched rST conversion of 'submitting-a-patch.rst', Kashyap Chamarthy, 2021/11/22
- Re: [PATCH 1/2] docs: Fix botched rST conversion of 'submitting-a-patch.rst', Thomas Huth, 2021/11/22
- Re: [PATCH 1/2] docs: Fix botched rST conversion of 'submitting-a-patch.rst', Kashyap Chamarthy, 2021/11/22
- Re: [PATCH 1/2] docs: Fix botched rST conversion of 'submitting-a-patch.rst', Peter Maydell, 2021/11/22
- Re: [PATCH 1/2] docs: Fix botched rST conversion of 'submitting-a-patch.rst', Thomas Huth, 2021/11/22
- Re: [PATCH 1/2] docs: Fix botched rST conversion of 'submitting-a-patch.rst', Daniel P . Berrangé, 2021/11/22
- Re: [PATCH 1/2] docs: Fix botched rST conversion of 'submitting-a-patch.rst', Thomas Huth, 2021/11/22
- Re: [PATCH 1/2] docs: Fix botched rST conversion of 'submitting-a-patch.rst',
Peter Maydell <=
- Re: [PATCH 1/2] docs: Fix botched rST conversion of 'submitting-a-patch.rst', Thomas Huth, 2021/11/22
- Re: [PATCH 1/2] docs: Fix botched rST conversion of 'submitting-a-patch.rst', Kashyap Chamarthy, 2021/11/22
Re: [PATCH 0/2] Fix broken rST conversion of "submitting-a-patch.rst", Eric Blake, 2021/11/19