Merge pull request #25 from clue-labs/php8.4

Run tests on PHP 8.4 and update test environment

Author: clue

.github/workflows/ci.yml

@@ -7,10 +7,11 @@ on:
 jobs:
   PHPUnit:
     name: PHPUnit (PHP ${{ matrix.php }})
-    runs-on: ubuntu-22.04
+    runs-on: ubuntu-24.04
     strategy:
       matrix:
         php:
+          - 8.4
           - 8.3
           - 8.2
           - 8.1
@@ -38,7 +39,7 @@ jobs:
 
   PHPUnit-hhvm:
     name: PHPUnit (HHVM)
-    runs-on: ubuntu-22.04
+    runs-on: ubuntu-24.04
     continue-on-error: true
     steps:
       - uses: actions/checkout@v4

README.md

@@ -183,7 +183,7 @@ composer install
 To run the test suite, go to the project root and run:
 
 ```bash
-php vendor/bin/phpunit
+vendor/bin/phpunit
 ```
 
 ## License

composer.json

@@ -10,16 +10,6 @@
             "email": "[email protected]"
         }
     ],
-    "autoload": {
-        "psr-4": {
-            "Clue\\React\\Multicast\\": "src/"
-        }
-    },
-    "autoload-dev": {
-        "psr-4": {
-            "Clue\\Tests\\React\\Multicast\\": "tests/"
-        }
-    },
     "require": {
         "php": ">=5.3",
         "react/datagram": "^1.10",
@@ -31,5 +21,15 @@
     },
     "suggest": {
         "ext-sockets": "Requires PHP 5.4+ and the low level socket API for listening on multicast addresses (socket options to send IGMP announcements)"
+    },
+    "autoload": {
+        "psr-4": {
+            "Clue\\React\\Multicast\\": "src/"
+        }
+    },
+    "autoload-dev": {
+        "psr-4": {
+            "Clue\\Tests\\React\\Multicast\\": "tests/"
+        }
     }
 }

src/Factory.php

@@ -22,12 +22,7 @@ class Factory
      * This value SHOULD NOT be given unless you're sure you want to explicitly use a
      * given event loop instance.
      *
-     * ```php
-     * $loop = React\EventLoop\Factory::create();
-     * $factory = new Factory($loop);
-     * ```
-     *
-     * @param LoopInterface $loop
+     * @param ?LoopInterface $loop
      */
     public function __construct($loop = null)
     {