Drop PHP 8.3

Author: mbabker

.github/workflows/run-tests.yml

@@ -9,11 +9,11 @@ jobs:
     strategy:
       fail-fast: true
       matrix:
-        php: ['8.3', '8.4', '8.5']
+        php: ['8.4', '8.5']
         composer-flags: ['--prefer-stable']
         can-fail: [false]
         include:
-          - php: '8.3'
+          - php: '8.4'
             composer-flags: '--prefer-stable --prefer-lowest'
             can-fail: false
 

.php-cs-fixer.php

@@ -1,12 +1,12 @@
 <?php declare(strict_types=1);
 
-return (new PhpCsFixer\Config())
+return new PhpCsFixer\Config()
     ->setRules([
         '@Symfony' => true,
         '@Symfony:risky' => true,
-        '@PHP83Migration' => true,
-        '@PHP82Migration:risky' => true,
-        '@PHPUnit100Migration:risky' => true,
+        '@PHP8x4Migration' => true,
+        '@PHP8x2Migration:risky' => true,
+        '@PHPUnit10x0Migration:risky' => true,
         'array_syntax' => ['syntax' => 'short'],
         'blank_line_after_opening_tag' => false,
         'declare_strict_types' => false,
@@ -18,7 +18,7 @@
     ])
     ->setRiskyAllowed(true)
     ->setFinder(
-        (new PhpCsFixer\Finder())
+        new PhpCsFixer\Finder()
             ->in(__DIR__.'/src')
             ->in(__DIR__.'/tests')
     );

composer.json

@@ -9,7 +9,7 @@
     ],
     "license": "MIT",
     "require": {
-        "php": "8.3.* || 8.4.* || 8.5.*",
+        "php": "8.4.* || 8.5.*",
         "guzzlehttp/psr7": "^2.4",
         "psr/http-message": "^1.0 || ^2.0",
         "ratchet/rfc6455": "^0.4",

docs/intro.md

@@ -10,4 +10,4 @@ The below table shows the support status and supported PHP versions for each maj
 
 | Version | Status         | PHP Versions |
 |---------|----------------|--------------|
-| 1.x     | In Development | 8.3-8.5      |
+| 1.x     | In Development | 8.4-8.5      |

src/Application.php

@@ -107,7 +107,7 @@ public function run(): void
 
         $socket = new SocketServer($this->uri, $this->context, $this->loop);
 
-        (new ReactPhpServer($middleware, $socket, $this->loop))->run();
+        new ReactPhpServer($middleware, $socket, $this->loop)->run();
     }
 
     public function route(string $path, MessageHandler|MessageMiddleware|string $handler, int $priority = 0): self

src/WAMP/Middleware/ParseWAMPMessage.php

@@ -110,7 +110,7 @@ public function onMessage(Connection $connection, string $data): void
                 $callID = array_shift($message);
                 $procURI = array_shift($message);
 
-                if (1 == \count($message) && \is_array($message[0])) {
+                if (1 === \count($message) && \is_array($message[0])) {
                     $message = $message[0];
                 }
 

tests/Http/GuzzleRequestParserTest.php

@@ -51,9 +51,10 @@ public function testConvertsToRequest(bool $valid, string $message): void
             $attributeStore->expects($this->once())
                 ->method('remove')
                 ->with('http.buffer');
-            $this->assertInstanceOf(RequestInterface::class, (new GuzzleRequestParser())->parse($connection, $message));
+
+            $this->assertInstanceOf(RequestInterface::class, new GuzzleRequestParser()->parse($connection, $message));
         } else {
-            $this->assertNotInstanceOf(RequestInterface::class, (new GuzzleRequestParser())->parse($connection, $message));
+            $this->assertNotInstanceOf(RequestInterface::class, new GuzzleRequestParser()->parse($connection, $message));
         }
     }
 
@@ -84,7 +85,7 @@ public function testRejectsARequestWhichIsTooBig(): void
             ->method('getAttributeStore')
             ->willReturn($attributeStore);
 
-        (new GuzzleRequestParser(10))->parse($connection, $message);
+        new GuzzleRequestParser(10)->parse($connection, $message);
     }
 
     public function testRejectsARequestWhichCannotBeParsed(): void
@@ -114,6 +115,6 @@ public function testRejectsARequestWhichCannotBeParsed(): void
             ->method('getAttributeStore')
             ->willReturn($attributeStore);
 
-        (new GuzzleRequestParser())->parse($connection, $message);
+        new GuzzleRequestParser()->parse($connection, $message);
     }
 }

tests/Http/Middleware/RejectBlockedIpAddressTest.php

@@ -198,7 +198,7 @@ public function testOnError(): void
 
     public function testManagesBlockedAddressList(): void
     {
-        $blockedAddresses = (new \ReflectionClass($this->middleware))->getProperty('blockedAddresses');
+        $blockedAddresses = new \ReflectionClass($this->middleware)->getProperty('blockedAddresses');
 
         $this->middleware->blockAddress('192.168.1.1');
         $this->middleware->blockAddress('192.168.0.0/24');

tests/Http/Middleware/RestrictToAllowedOriginsTest.php

@@ -274,7 +274,7 @@ public function testOnError(): void
 
     public function testManagesAllowedOriginList(): void
     {
-        $allowedOrigins = (new \ReflectionClass($this->middleware))->getProperty('allowedOrigins');
+        $allowedOrigins = new \ReflectionClass($this->middleware)->getProperty('allowedOrigins');
 
         $this->middleware->allowOrigin('192.168.1.1');
         $this->middleware->allowOrigin('localhost');

tests/ReactPhpServerTest.php

@@ -26,7 +26,7 @@ public static function setUpBeforeClass(): void
 
     public static function tearDownAfterClass(): void
     {
-        (new \ReflectionClass(Loop::class))->setStaticPropertyValue('instance', null);
+        new \ReflectionClass(Loop::class)->setStaticPropertyValue('instance', null);
     }
 
     protected function setUp(): void