Merge pull request php-curl-class#696 from zachborboa/master

Add static analysis checks to continuous integration

Author: zachborboa

.github/workflows/ci.yml

@@ -44,3 +44,7 @@ jobs:
         env:
           CI_PHP_VERSION: ${{ matrix.php }}
           CI_PHP_FUTURE_RELEASE: ${{ matrix.future-release }}
+
+      - name: Static analysis
+        run: "vendor/bin/psalm"
+        if: ${{ !matrix.future-release && matrix.php >= 7.1 }}

composer.json

@@ -21,7 +21,8 @@
         "ext-gd": "*",
         "phpcompatibility/php-compatibility": "*",
         "phpunit/phpunit": "*",
-        "squizlabs/php_codesniffer": "*"
+        "squizlabs/php_codesniffer": "*",
+        "vimeo/psalm": "*"
     },
     "suggest": {
         "ext-mbstring": "*"

psalm.xml

@@ -0,0 +1,27 @@
+<?xml version="1.0"?>
+<psalm
+    errorLevel="5"
+    resolveFromConfigFile="true"
+    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+    xmlns="https://getpsalm.org/schema/config"
+    xsi:schemaLocation="https://getpsalm.org/schema/config vendor/vimeo/psalm/config.xsd"
+>
+    <projectFiles>
+        <directory name="src" />
+        <ignoreFiles>
+            <directory name="vendor" />
+        </ignoreFiles>
+    </projectFiles>
+    <issueHandlers>
+        <UndefinedClass>
+            <errorLevel type="suppress">
+                <referencedClass name="CurlHandle" />
+                <file name="src/Curl/Curl.php" />
+            </errorLevel>
+            <errorLevel type="suppress">
+                <referencedClass name="CurlMultiHandle" />
+                <file name="src/Curl/MultiCurl.php" />
+            </errorLevel>
+        </UndefinedClass>
+    </issueHandlers>
+</psalm>

src/Curl/Curl.php

@@ -1428,7 +1428,7 @@ public function removeHeader($key)
      *
      * @access public
      * @param  bool $on
-     * @param  resource $output
+     * @param  resource|string $output
      */
     public function verbose($on = true, $output = 'STDERR')
     {