sirbrillig / phpcs-variable-analysis

Find undefined and unused variables with the PHP Codesniffer static analysis tool.
Other
135 stars 14 forks source link

GH Actions: various tweaks / PHP 8.2 not allowed to fail #285

Closed jrfnl closed 1 year ago

jrfnl commented 1 year ago

GH Actions: use PHP latest

... for those tasks where the PHP version isn't (or shouldn't be) that relevant.

Includes updating the task name to not include the PHP version (latest will roll on automatically).

GH Actions: minor simplification

... of the bash date command in the earlier pulled cache busting.

GH Actions: update PHP versions in workflows

PHP 8.2 has been released today 🎉 and the setup-php action has announced support for PHP 8.3, so adding PHP 8.3 to the matrix and no longer allowing PHP 8.2 to fail the build.

Note: PHPCS does not (yet) have full syntax support for PHP 8.2, but it does have runtime support (for the most part, see squizlabs/PHP_CodeSniffer#3629).

Builds against PHP 8.3 are still allowed to fail for now.

jrfnl commented 1 year ago

Note: looks like the latest version of Psalm is throwing two new issues. As far as I can see that's unrelated to this PR and needs a doc fix.

sirbrillig commented 1 year ago

https://github.com/sirbrillig/phpcs-variable-analysis/pull/286 should fix the Psalm issue.