[edk2-devel] [edk2-redfish-client][PATCH] RedfishClientPkg/.github: fix uncrustify check error

[Chang, Abner via groups.io]

[AMD Official Use Only - General]

Reviewed-by: Abner Chang <abner.chang at amd.com>

> -----Original Message-----
> From: Nickle Wang <nicklew at nvidia.com>
> Sent: Tuesday, June 13, 2023 9:32 PM
> To: devel at edk2.groups.io
> Cc: Chang, Abner <Abner.Chang at amd.com>; Igor Kulchytskyy
> <igork at ami.com>
> Subject: [edk2-redfish-client][PATCH] RedfishClientPkg/.github: fix uncrustify
> check error
>
> Caution: This message originated from an External Source. Use proper caution
> when opening attachments, clicking links, or responding.
>
>
> Only run uncrustify check to c file and h file in script.
>
> Signed-off-by: Nickle Wang <nicklew at nvidia.com>
> Cc: Abner Chang <abner.chang at amd.com>
> Cc: Igor Kulchytskyy <igork at ami.com>
> ---
>  .github/workflows/uncrustify-check.sh | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/.github/workflows/uncrustify-check.sh
> b/.github/workflows/uncrustify-check.sh
> index 59ebc0dc..ff61a6ae 100755
> --- a/.github/workflows/uncrustify-check.sh
> +++ b/.github/workflows/uncrustify-check.sh
> @@ -31,7 +31,13 @@ fi
>  cd "$REPO_PATH"
>
>  FAILURE=0
> -CHANGED_FILES=$(git diff --name-only HEAD~$NO_COMMITS)
> +CHANGED_FILES=$(git diff --name-only HEAD~$NO_COMMITS | grep
> '\(\.c$\|\.h$\)')
> +if [ -z "$CHANGED_FILES" ]
> +then
> +  echo "No c or h file to run uncrustify check"
> +  exit 0
> +fi
> +
>  for file in $CHANGED_FILES
>  do
>    echo "Uncrustify check file: $file"
> --
> 2.17.1



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