[edk2-devel] [PATCH v1 0/8] Fix new typos reported

Ard Biesheuvel ardb at kernel.org
Tue May 17 17:31:59 UTC 2022


On Tue, 17 May 2022 at 18:25, Michael Kubacki
<mikuback at linux.microsoft.com> wrote:
>
> Hi Ard,
>
> I think that's a reasonable approach.
>
> We could also consider locking onto a specific cspell version to
> decrease the likelihood of this sporadically appearing in the future.
>
> In this case, I would prefer not to make the decision to disable spell
> check entirely on behalf of various package maintainers though. I'm just
> trying to keep the status quo from unblocking other changes.
>
> Do you think that's something you or others could add as a change on top
> of this series?
>

I guess that could wait until after the stable tag. However, your
current series adds words such as "addressig" and "characteritics" to
the ignorelist of ArmPkg, and that is something I do have a problem
with: this is just busywork, as there is no confusion whatsoever about
the meaning of the texts in question, nobody is proposing changes to
the code that contains these typos, and adding typos to these
ignorelists is clearly not the correct solution.

CI and automation are fine, but here, they are just creating problems,
resulting in pointless churn, and impeding the stable tag process.
Please, just turn it off.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#89851): https://edk2.groups.io/g/devel/message/89851
Mute This Topic: https://groups.io/mt/91166946/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-




More information about the edk2-devel-archive mailing list