Re: 回复: [edk2-devel] [PATCH] MdePkg Include: Fix typos

2023-05-22 Thread Michael Kubacki
u ; Andrew Fish ; Giri Mudusuru 主题: Re: [edk2-devel] [PATCH] MdePkg Include: Fix typos Thanks Michael for below info. On May 18, 2023, at 8:36 AM, Michael Kubacki wrote: The spell check CI plugin is in "AuditOnly" mode so it is not actually failing the build. https://githu

回复: [edk2-devel] [PATCH] MdePkg Include: Fix typos

2023-05-21 Thread gaoliming via groups.io
ming ; > Michael D Kinney ; Zhiguang Liu > ; Andrew Fish ; Giri Mudusuru > > 主题: Re: [edk2-devel] [PATCH] MdePkg Include: Fix typos > > Thanks Michael for below info. > > > > On May 18, 2023, at 8:36 AM, Michael Kubacki > wrote: > > > > The spe

Re: [edk2-devel] [PATCH] MdePkg Include: Fix typos

2023-05-20 Thread Giri Mudusuru via groups.io
Thanks Michael for below info. > On May 18, 2023, at 8:36 AM, Michael Kubacki > wrote: > > The spell check CI plugin is in "AuditOnly" mode so it is not actually > failing the build. > > https://github.com/tianocore/edk2/blob/master/MdePkg/MdePkg.ci.yaml#L166 > > This has more info about ho

Re: [edk2-devel] [PATCH] MdePkg Include: Fix typos

2023-05-17 Thread Michael Kubacki
The spell check CI plugin is in "AuditOnly" mode so it is not actually failing the build. https://github.com/tianocore/edk2/blob/master/MdePkg/MdePkg.ci.yaml#L166 This has more info about how to configure the plugin in each package CI YAML file - https://github.com/tianocore/edk2/tree/master/

Re: [edk2-devel] [PATCH] MdePkg Include: Fix typos

2023-05-17 Thread Giri Mudusuru via groups.io
Hi Liming, Most of the errors except below macro are in comments. Not sure if CI SpellCheck is skipping comments? I can revert the macro fix from this patch. I noticed that it is duplicated in Acpi.h and Acpi10.h. Thanks, -Giri > On May 16, 2023, at 6:49 AM, gaoliming wrote: > > Giri: > Edk2

Re: [edk2-devel] [PATCH] MdePkg Include: Fix typos

2023-05-14 Thread Giri Mudusuru via groups.io
Hi Liming, These typos are found during reviews, sync with open source and other debug. Just kept running list and then a simple search and replace in repo. I have fix for other packages too, just starting with MdePkg to get some feedback on size of patch (number of files etc) I am using VS Co

[edk2-devel] [PATCH] MdePkg Include: Fix typos

2023-05-12 Thread Giri Mudusuru via groups.io
compatability->compatibility EFI_MEDIA_CHNAGED->EFI_MEDIA_CHANGED Funtion->Function exhausive->exhaustive Propery->Property StartAdress->StartAddress sucessful->successful writting->writing Cc: Michael D Kinney Cc: Liming Gao Cc: Zhiguang Liu Cc: Andrew Fish Signed-off-by: Giri Mudusuru ---