On Thu, 15 Dec 2022 at 00:21, Michael Kubacki
<mikub...@linux.microsoft.com> wrote:
>
> Yes. It will also reduce frequency of incoming patches that must be
> reviewed and merged due to people continuously fixing trivial spelling
> errors.
>

In that case, NAK to this patch (and the ArmVirtPkg one). Unless we
add a button to the GitHub UI that permits me to override a negative
CI result on a PR.

> On 12/14/2022 6:07 PM, Ard Biesheuvel wrote:
> > On Wed, 14 Dec 2022 at 23:53, <mikub...@linux.microsoft.com> wrote:
> >>
> >> From: Michael Kubacki <michael.kuba...@microsoft.com>
> >>
> >> Audit mode was enabled for the spellcheck CI plugin. It is no longer
> >> needed with recent changes. Spelling errors can be checked in the
> >> package moving forward.
> >>
> >> Cc: Leif Lindholm <quic_llind...@quicinc.com>
> >> Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org>
> >> Cc: Sami Mujawar <sami.muja...@arm.com>
> >> Signed-off-by: Michael Kubacki <michael.kuba...@microsoft.com>
> >
> > Will this patch result in PRs potentially being rejected by pre-merge
> > CI due to trivial spelling errors?
> >
> >
> >> ---
> >>   ArmPkg/ArmPkg.ci.yaml | 3 ++-
> >>   1 file changed, 2 insertions(+), 1 deletion(-)
> >>
> >> diff --git a/ArmPkg/ArmPkg.ci.yaml b/ArmPkg/ArmPkg.ci.yaml
> >> index c8dface6821a..a304c7966cf7 100644
> >> --- a/ArmPkg/ArmPkg.ci.yaml
> >> +++ b/ArmPkg/ArmPkg.ci.yaml
> >> @@ -87,7 +87,7 @@
> >>
> >>       ## options defined .pytool/Plugin/SpellCheck
> >>       "SpellCheck": {
> >> -        "AuditOnly": True,
> >> +        "AuditOnly": False,
> >>           "IgnoreFiles": [
> >>               "Library/ArmSoftFloatLib/berkeley-softfloat-3/**"
> >>           ],                           # use gitignore syntax to ignore 
> >> errors
> >> @@ -148,6 +148,7 @@
> >>             "fcmplt",
> >>             "ffreestanding",
> >>             "frsub",
> >> +          "hauser",
> >>             "hisilicon",
> >>             "iccabpr",
> >>             "iccbpr",
> >> --
> >> 2.28.0.windows.1
> >>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#97403): https://edk2.groups.io/g/devel/message/97403
Mute This Topic: https://groups.io/mt/95678218/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to