diff --git a/.github/workflows/phpcs.yml b/.github/workflows/phpcs.yml index 18f0ef7..84a8a39 100644 --- a/.github/workflows/phpcs.yml +++ b/.github/workflows/phpcs.yml @@ -63,7 +63,7 @@ jobs: run: ./vendor/bin/phpunit-coverage-check -t 80 clover.xml - name: Integration tests - if: ${{ (!cancelled()) && (runner.os == 'ubuntu-latest') }} + if: ${{ !cancelled() && runner.os == 'ubuntu-latest' }} run: | # There is one failure (exit with error 2, because some are fixable). vendor/bin/phpcs --standard=moodle moodle/Tests/fixtures/integration_test_ci.php | tee output.txt @@ -72,7 +72,7 @@ jobs: # The failure is fixed (exit with error 1, because all fixable ones were fixed). vendor/bin/phpcbf --standard=moodle moodle/Tests/fixtures/integration_test_ci.php | tee output.txt - [[ "${PIPESTATUS[0]}" = 1 ]] + [[ "${PIPESTATUS[0]}" = 0 ]] grep -q "A TOTAL OF 1 ERROR WERE FIXED IN 1 FILE" output.txt # So, there isn't any failure any more (exit without error, aka, 0)