Fix PHP 8.4 deprecation warnings (#109)

Codeception · Dec 8, 2024 · 03af8d4 · 03af8d4
1 parent 108b200
commit 03af8d4
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -8,11 +8,11 @@ jobs:
 
     strategy:
       matrix:
-        php: [8.0, 8.1, 8.2]
+        php: [8.0, 8.1, 8.2, 8.3, 8.4]
 
     steps:
     - name: Checkout code
-      uses: actions/checkout@v3
+      uses: actions/checkout@v4
 
     - name: Setup PHP
       uses: shivammathur/setup-php@v2

diff --git a/src/Codeception/Lib/Connector/Yii2.php b/src/Codeception/Lib/Connector/Yii2.php
@@ -250,7 +250,7 @@ public function getCsrfParamName(): string
         return $this->getApplication()->request->csrfParam;
     }
 
-    public function startApp(\yii\log\Logger $logger = null): void
+    public function startApp(?\yii\log\Logger $logger = null): void
     {
         codecept_debug('Starting application');
         $config = require($this->configFile);

diff --git a/src/Codeception/Module/Yii2.php b/src/Codeception/Module/Yii2.php
@@ -723,11 +723,11 @@ public function grabComponent(string $component): null|object
      * $I->seeEmailIsSent(3);
      * ```
      *
-     * @param int $num
+     * @param int|null $num
      * @throws \Codeception\Exception\ModuleException
      * @part email
      */
-    public function seeEmailIsSent(int $num = null): void
+    public function seeEmailIsSent(?int $num = null): void
     {
         if ($num === null) {
             $this->assertNotEmpty($this->grabSentEmails(), 'emails were sent');