Drop PHP 7.3

Author: dvdoug

.github/workflows/ci.yml

@@ -20,7 +20,6 @@ jobs:
           - "8.1"
           - "8.0"
           - "7.4"
-          - "7.3"
 
         php-code-coverage-version:
           - "^10.0"
@@ -43,14 +42,10 @@ jobs:
         exclude:
           - php-version: "7.4"
             symfony-version: "^6.0"
-          - php-version: "7.3"
-            symfony-version: "^6.0"
           - php-version: "8.0"
             php-code-coverage-version: "^10.0"
           - php-version: "7.4"
             php-code-coverage-version: "^10.0"
-          - php-version: "7.3"
-            php-code-coverage-version: "^10.0"
 
     steps:
     - uses: actions/checkout@v4

composer.json

@@ -12,7 +12,7 @@
         }
     ],
     "require": {
-        "php": "^7.3 || ^8.0",
+        "php": "^7.4 || ^8.0",
         "composer-runtime-api": "^2.0",
         "behat/behat": "^3.10",
         "composer/semver": "^3.0",

src/Controller/Cli/CodeCoverageController.php

@@ -14,16 +14,12 @@
 
 class CodeCoverageController implements Controller
 {
-    /**
-     * {@inheritdoc}
-     */
     public function configure(Command $command): void
     {
         $command->addOption('no-coverage', null, InputOption::VALUE_NONE, 'Skip Code Coverage generation');
     }
 
     /**
-     * {@inheritdoc}
      * @codeCoverageIgnore
      */
     public function execute(InputInterface $input, OutputInterface $output): void

src/Extension.php

@@ -37,16 +37,10 @@
 
 class Extension implements ExtensionInterface
 {
-    /**
-     * {@inheritdoc}
-     */
     public function initialize(ExtensionManager $extensionManager): void
     {
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function load(ContainerBuilder $container, array $config): void
     {
         $container->registerForAutoconfiguration(Controller::class)->addTag(CliExtension::CONTROLLER_TAG);
@@ -61,9 +55,6 @@ public function load(ContainerBuilder $container, array $config): void
         $container->setParameter('behat.code_coverage.config.cache', $config['cache']);
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function configure(ArrayNodeDefinition $builder): void
     {
         $builder
@@ -185,17 +176,11 @@ public function configure(ArrayNodeDefinition $builder): void
         ->end();
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getConfigKey()
     {
         return 'code_coverage';
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function process(ContainerBuilder $container): void
     {
         /** @var InputInterface $input */

src/Service/ReportService.php

@@ -24,10 +24,7 @@
 
 class ReportService
 {
-    /**
-     * @var array
-     */
-    private $config;
+    private array $config;
 
     /**
      * Constructor.

src/Subscriber/EventSubscriber.php

@@ -15,17 +15,11 @@
 
 class EventSubscriber implements EventSubscriberInterface
 {
-    /**
-     * @var CodeCoverage
-     */
-    private $coverage;
+    private ?CodeCoverage $coverage;
 
-    /**
-     * @var ReportService
-     */
-    private $reportService;
+    private ReportService $reportService;
 
-    public function __construct(ReportService $reportService, CodeCoverage $coverage = null)
+    public function __construct(ReportService $reportService, ?CodeCoverage $coverage)
     {
         $this->reportService = $reportService;
         $this->coverage = $coverage;

tests/EventSubscriberTest.php

@@ -27,7 +27,7 @@ class EventSubscriberTest extends TestCase
 {
     public function testCanSubscribeToEvents(): void
     {
-        $eventSubscriber = new EventSubscriber(new ReportService([]));
+        $eventSubscriber = new EventSubscriber(new ReportService([]), null);
         $dispatcher = new EventDispatcher();
         $dispatcher->addSubscriber($eventSubscriber);
 
@@ -40,7 +40,7 @@ public function testBeforeScenarioWithNoCoverage(): void
     {
         $event = $this->createMock(ScenarioTested::class);
 
-        $subscriber = new EventSubscriber(new ReportService([]));
+        $subscriber = new EventSubscriber(new ReportService([]), null);
         $subscriber->beforeScenario($event);
 
         self::assertTrue(true);
@@ -50,7 +50,7 @@ public function testAfterScenarioWithNoCoverage(): void
     {
         $event = $this->createMock(ScenarioTested::class);
 
-        $subscriber = new EventSubscriber(new ReportService([]));
+        $subscriber = new EventSubscriber(new ReportService([]), null);
         $subscriber->afterScenario($event);
 
         self::assertTrue(true);
@@ -60,7 +60,7 @@ public function testAfterExerciseWithNoCoverage(): void
     {
         $event = $this->createMock(ExerciseCompleted::class);
 
-        $subscriber = new EventSubscriber(new ReportService([]));
+        $subscriber = new EventSubscriber(new ReportService([]), null);
         $subscriber->afterExercise($event);
 
         self::assertTrue(true);