Ard, Why would you want to do that? The whole point of CI is to establish a minimum quality level for all code in the project. They can be disabled with updates to the YAML file. Checks can be

<mikuback@...> wrote: The only thing I'd like to know is how I can override the CI and merge a PR that was rejected by the checks you are enabling here.

Hi Ard, I haven't seen any action items for the v1 series. Can you please check the series again and let me know if you have any further concerns? Thanks, Michael

Hi Chasel, Your CI YAML file feedback in v1 is addressed now in v2. Can you please provide your review on this patch and [PATCH v2 5/6]? Note that I updated the commit message for this patch to

Hi Jian, Can you please provide a review for this patch? Mike Kinney has already given an R-b for [PATCH v1 2/2] so the series will be ready once this patch is reviewed. Thanks, Michael

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D4048 Adds these packages to a new edk2 matrix job so they can be validated in edk2 CI. Cc:

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D4048 Adds IntelFsp2WrapperPkg to the list of supported build packages for edk2 CI and

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D4048 This package did not have CI enabled so code changes were merged that fail uncrustify

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D4048 Adds IntelFsp2Pkg to the list of supported build packages for edk2 CI and defines an

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D4048 The FILE_GUID for this library instance file is a duplicate

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D4048 This package did not have CI enabled so code changes were merged that fail uncrustify

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D4048 - Enables CI in IntelFsp2Pkg and IntelFsp2WrapperPkg. - Fixes several pre-existing

Thanks Leif. I will fix as your suggestion. -Nhi

Pushed as 8d8bb28d2c..17a682058c

Thanks for this. Reviewed-by: Leif Lindholm <quic_llindhol@...>

Patch merged: https://github.com/tianocore/edk2/commit/f46c7d1e36c901e72cc0daee8429275bcb319eb9 Thanks, Chasel

Reviewed-by: Andrew Fish <afish@...>

Apologies for the delay. Merged as f4679715a40d..8d8bb28d2c4b Regards, Sami Mujawar

Reviewed-by: Michael D Kinney <michael.d.kinney@...>

Patch merged: https://github.com/tianocore/edk2/commit/981bf66d5a7b1a31be5ffccc15fe13b97300e124 Thanks, Chasel