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.)