Skip to content

fix cs and phpstan

fix cs and phpstan #22

Triggered via push November 16, 2024 12:25
Status Failure
Total duration 37s
Billable time 1m
Artifacts

stan.yaml

on: push
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 1 warning
phpstan: app/Http/Controllers/Api/V1/SiteController.php#L39
Parameter #1 $baseUrl of class App\RemoteSite\Connection constructor expects string, mixed given.
phpstan: app/Http/Controllers/Api/V1/SiteController.php#L39
Parameter #2 $key of class App\RemoteSite\Connection constructor expects string, mixed given.
phpstan: app/Http/Controllers/Api/V1/SiteController.php#L53
Property App\Models\Site::$key (string) does not accept mixed.
phpstan: app/Http/Controllers/Api/V1/SiteController.php#L54
Property App\Models\Site::$url (string) does not accept mixed.
phpstan: app/Http/Resources/Site.php#L17
Method App\Http\Resources\Site::toArray() should return array<string, mixed> but returns array|Illuminate\Contracts\Support\Arrayable|JsonSerializable.
phpstan: app/Http/Resources/SiteCollection.php#L17
Method App\Http\Resources\SiteCollection::toArray() should return array<int|string, mixed> but returns array|Illuminate\Contracts\Support\Arrayable|JsonSerializable.
phpstan: app/Jobs/CheckSiteHealth.php#L8
Class App\RemoteSite\Connection referenced with incorrect case: app\Remotesite\Connection.
phpstan: app/Jobs/CheckSiteHealth.php#L30
Class App\RemoteSite\Connection referenced with incorrect case: app\Remotesite\Connection.
phpstan: app/Jobs/CheckSiteHealth.php#L34
Parameter #1 $value of function collect expects Illuminate\Contracts\Support\Arrayable<(int|string), mixed>|iterable<(int|string), mixed>|null, App\RemoteSite\Responses\HealthCheck given.
phpstan: app/Jobs/CheckSiteHealth.php#L34
Unable to resolve the template type TKey in call to function collect
phpstan
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/