Merge pull request #63 from Howriq/issue-61

Issue #61: Increase PHPstan to 6 , then 7, then 8 , then 9

Author: arhimede

README.md

@@ -22,7 +22,9 @@ Documentation is available at: https://docs.dotkernel.org/light-documentation/
 [![Continuous Integration](https://github.com/dotkernel/light/actions/workflows/continuous-integration.yml/badge.svg?branch=1.0)](https://github.com/dotkernel/light/actions/workflows/continuous-integration.yml)
 [![codecov](https://codecov.io/gh/dotkernel/light/graph/badge.svg?token=UpVJ5ELvfZ)](https://codecov.io/gh/dotkernel/light)
 [![Qodana](https://github.com/dotkernel/light/actions/workflows/qodana_code_quality.yml/badge.svg)](https://github.com/dotkernel/light/actions/workflows/qodana_code_quality.yml)
+
 [![PHPStan](https://github.com/dotkernel/light/actions/workflows/static-analysis.yml/badge.svg?branch=1.0)](https://github.com/dotkernel/light/actions/workflows/static-analysis.yml)
+![PHPstan Level](https://img.shields.io/badge/PHPStan-level%208-brightgreen)
 
 ## Installing Dotkernel `light`
 

bin/composer-post-install-script.php

@@ -4,6 +4,9 @@
 
 // phpcs:disable PSR1.Files.SideEffects.FoundWithSymbols
 
+/**
+ * @param array{source: string, destination: string} $file
+ */
 function copyFile(array $file): void
 {
     if (is_readable($file['destination'])) {

phpstan.neon

@@ -1,7 +1,7 @@
 includes:
     - vendor/phpstan/phpstan-phpunit/extension.neon
 parameters:
-    level: 5
+    level: 8
     paths:
         - bin
         - config

src/App/src/ConfigProvider.php

@@ -10,6 +10,12 @@
 
 class ConfigProvider
 {
+    /**
+    @return array{
+     *     dependencies: array<mixed>,
+     *     templates: array<mixed>,
+     * }
+     */
     public function __invoke(): array
     {
         return [
@@ -18,6 +24,12 @@ public function __invoke(): array
         ];
     }
 
+    /**
+     * @return array{
+     *     delegators: array<class-string, array<class-string>>,
+     *     factories: array<class-string, class-string>,
+     * }
+     */
     public function getDependencies(): array
     {
         return [
@@ -32,6 +44,16 @@ public function getDependencies(): array
         ];
     }
 
+    /**
+     * @return array{
+     *     paths: array{
+     *          app: array{literal-string&non-falsy-string},
+     *          error: array{literal-string&non-falsy-string},
+     *          layout: array{literal-string&non-falsy-string},
+     *          partial: array{literal-string&non-falsy-string},
+     *     }
+     * }
+     */
     public function getTemplates(): array
     {
         return [

src/Page/src/ConfigProvider.php

@@ -13,6 +13,12 @@
 
 class ConfigProvider
 {
+    /**
+    @return array{
+     *     dependencies: array<mixed>,
+     *     templates: array<mixed>,
+     * }
+     */
     public function __invoke(): array
     {
         return [
@@ -21,6 +27,13 @@ public function __invoke(): array
         ];
     }
 
+    /**
+    @return array{
+     *     delegators: array<class-string, array<class-string>>,
+     *     factories: array<class-string, class-string>,
+     *     aliases: array<class-string, class-string>
+     * }
+     */
     public function getDependencies(): array
     {
         return [
@@ -39,6 +52,11 @@ public function getDependencies(): array
         ];
     }
 
+    /**
+    @return array{
+     *     paths: array{page: array{literal-string&non-falsy-string}}
+     * }
+     */
     public function getTemplates(): array
     {
         return [

test/Unit/App/ConfigProviderTest.php

@@ -10,6 +10,7 @@
 
 class ConfigProviderTest extends TestCase
 {
+    /** @var array<string, array<string, mixed>>*/
     protected array $config = [];
 
     protected function setup(): void

test/Unit/Page/ConfigProviderTest.php

@@ -13,6 +13,7 @@
 
 class ConfigProviderTest extends TestCase
 {
+    /** @var array<string, array<string, mixed>>*/
     protected array $config = [];
 
     protected function setup(): void