⬆️ Upgrade AlpineJS monorepo to v3.14.7 (patch) #28
phpinsights.yml
on: pull_request
PHP Insights checks
21s
Annotations
10 errors and 1 warning
PHP Insights checks:
site/config/config.php#L68
* [Unused variable] Unused variable $text.
* [Assignment in condition] Assignment in if condition is not allowed.
|
PHP Insights checks:
site/config/config.php#L69
* [Assignment in condition] Assignment in if condition is not allowed.
|
PHP Insights checks:
site/config/config.php#L21
* [Static closure] Closure not using "$this" should be declared static.
|
PHP Insights checks:
site/config/config.php#L39
* [Static closure] Closure not using "$this" should be declared static.
|
PHP Insights checks:
site/config/config.php#L61
* [Static closure] Closure not using "$this" should be declared static.
|
PHP Insights checks:
site/config/config.php#L90
* [Static closure] Closure not using "$this" should be declared static.
|
PHP Insights checks:
site/config/config.php#L108
* [Static closure] Closure not using "$this" should be declared static.
|
PHP Insights checks:
site/config/config.php#L1
* [Void return] @@ -20,3 +20,3 @@
'hooks' => [
- 'system.exception' => function ($exception) {
+ 'system.exception' => function ($exception): void {
$dsn = option('sentry.dsn');
|
PHP Insights checks:
phpinsights.php#L75
* [Array indent] Array key not indented correctly; expected 8 spaces but found 10
* [Trailing array comma] Multi-line arrays must have a trailing comma after the last element.
|
PHP Insights checks:
site/snippets/layout.php#L44
* [Array indent] Array key not indented correctly; expected 4 spaces but found 16
|
PHP Insights checks
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|