code review changes

Author: nikow13

.github/workflows/build.yml

@@ -9,11 +9,11 @@ on:
 
 jobs:
   build:
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
     strategy:
       fail-fast: false
       matrix:
-        php-versions: [ '8.2', '8.3', '8.4']
+        php-versions: [ '8.0', '8.1', '8.2', '8.3', '8.4']
         coverage-driver: [ 'pcov' ]
 
     steps:
@@ -58,15 +58,15 @@ jobs:
         token: ${{ secrets.CODECOV_TOKEN }}
 
   phar:
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
     needs: build
     steps:
       - uses: actions/checkout@v3
 
       - name: Install PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: 8.4
+          php-version: 8.0
           tools: composer:v2.8
         env:
           GITHUB_TOKEN: ${{ secrets.TOKEN_GITHUB }}
@@ -92,10 +92,10 @@ jobs:
 
   smoke-test-phar:
     needs: phar
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
     strategy:
       matrix:
-        php-versions: [ '8.2', '8.3', '8.4' ]
+        php-versions: [ '8.0', '8.1', '8.2', '8.3', '8.4' ]
 
     steps:
       - name: Install PHP
@@ -116,7 +116,7 @@ jobs:
 
   publish_phar:
     needs: [build, smoke-test-phar]
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
     if: github.event_name == 'release'
     steps:
       - name: "Download phar ./phparkitect-${{ github.sha }}.phar"
@@ -135,7 +135,7 @@ jobs:
 
   publish_docker_images:
     needs: [build, smoke-test-phar]
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
 
     if: github.ref == 'refs/heads/main' || github.event_name == 'release'
     steps:

Dockerfile

@@ -1,4 +1,4 @@
-ARG PHP_VERSION=8.4
+ARG PHP_VERSION=8.0
 
 FROM php:${PHP_VERSION}-cli-alpine AS php_build
 

src/CLI/Progress/ProgressBarProgress.php

@@ -15,7 +15,7 @@
  */
 class ProgressBarProgress implements Progress
 {
-    private \Symfony\Component\Console\Output\NullOutput|OutputInterface $output;
+    private OutputInterface $output;
 
     private ProgressBar $progress;
 

src/CLI/TargetPhpVersion.php

@@ -9,10 +9,6 @@
 class TargetPhpVersion
 {
     public const VALID_PHP_VERSIONS = [
-        '7.1',
-        '7.2',
-        '7.3',
-        '7.4',
         '8.0',
         '8.1',
         '8.2',

tests/Unit/Expressions/ForClasses/ResideInOneOfTheseNamespacesTest.php

@@ -30,10 +30,6 @@ public static function shouldMatchNamespacesProvider(): array
 
     /**
      * @dataProvider shouldMatchNamespacesProvider
-     *
-     * @param mixed $expectedNamespace
-     * @param mixed $actualFQCN
-     * @param mixed $explanation
      */
     public function test_it_should_match_namespace_and_descendants(string $expectedNamespace, string $actualFQCN, string $explanation): void
     {