From c0938341337e7ab0049ebbe62b27ce3a22faa7d8 Mon Sep 17 00:00:00 2001 From: Simon Sprankel Date: Wed, 16 Aug 2023 13:35:39 +0200 Subject: [PATCH] Only execute phpcs if changes exist (#101) --- coding-standard-baseline/action.yml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/coding-standard-baseline/action.yml b/coding-standard-baseline/action.yml index cca387c..5311e15 100755 --- a/coding-standard-baseline/action.yml +++ b/coding-standard-baseline/action.yml @@ -83,11 +83,11 @@ runs: shell: bash run: | sed "s/ /\n/g" ${{ github.workspace }}/modified_files.txt > ${{ github.workspace }}/newline_file.txt - grep -iE "\.php|\.phtml|\.html|\.xml" ${{ github.workspace }}/newline_file.txt > ${{ github.workspace }}/phpcs_files.txt || exit 0 + grep -iE "\.php|\.phtml|\.html|\.xml" ${{ github.workspace }}/newline_file.txt > ${{ github.workspace }}/phpcs_files.txt || /bin/true - name: Checkout - Before Merge shell: bash - if: ${{ hashFiles('phpcs_files.txt') != '' }} + if: test -s ${{ github.workspace }}/phpcs_files.txt run: | if ${{ github.event_name == 'pull_request' }}; then git checkout ${{ github.event.pull_request.base.ref }} @@ -97,7 +97,7 @@ runs: - name: Filter php files and execute phpcs - Before Merge shell: bash - if: ${{ hashFiles('phpcs_files.txt') != '' }} + if: test -s ${{ github.workspace }}/phpcs_files.txt run: | php vendor/bin/phpcs --standard=Magento2 \ $([ -n "${{ inputs.severity }}" ] && echo "--severity=${{ inputs.severity }}") \ @@ -107,7 +107,7 @@ runs: - name: Checkout after Merge and execute phpcs shell: bash - if: ${{ hashFiles('phpcs_files.txt') != '' }} + if: test -s ${{ github.workspace }}/phpcs_files.txt run: | if ${{ github.event_name == 'pull_request' }}; then git checkout ${{ github.event.pull_request.head.ref }}