Commit 89c901b

committed

GH Actions: harden the workflow against PHPCS ruleset errors

If there is a ruleset error, the `cs2pr` action doesn't receive an `xml` report and exits with a `0` error code, even though the PHPCS run failed (though not on CS errors, but on a ruleset error). This changes the GH Actions workflow to allow for that situation and still fail the build in that case.

1 parent a829c57 commit 89c901bCopy full SHA for 89c901b

File tree

1 file changed

-0

lines changed

.github/workflows
- check-cs.yml

1 file changed

-0

lines changed

`‎.github/workflows/check-cs.yml`

Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,9 @@ jobs:`
`40`	`40`	`composer update --prefer-dist --no-suggest --no-progress --no-interaction`
`41`	`41`
`42`	`42`	`- name: Check Code Style`
	`43`	`+ id: phpcs`
`43`	`44`	`run: vendor/bin/phpcs --report-full --report-checkstyle=./phpcs-report.xml`
`44`	`45`
`45`	`46`	`- name: Show PHPCS results in PR`
	`47`	`+ if: ${{ always() && steps.phpcs.outcome == 'failure' }}`
`46`	`48`	`run: cs2pr ./phpcs-report.xml`

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit 89c901b

File tree

1 file changed

1 file changed

`‎.github/workflows/check-cs.yml`

0 commit comments