Merge pull request #24 from jonnyeom/4.x/update-standards

4.x/update standards
unleashedtech · Jan 26, 2024 · c5a3e44 · c5a3e44
2 parents 9e69e62 + 4f1d013
commit c5a3e44
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 17 deletions.
diff --git a/.docker/validate-against-schema/Dockerfile b/.docker/validate-against-schema/Dockerfile
@@ -1,3 +1,3 @@
-FROM alpine:3.11.3
+FROM alpine:3.17.3
 
-RUN apk add --no-cache libxml2-utils
+RUN apk add --no-cache libxml2-utils=2.10.4-r0
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -15,12 +15,12 @@ jobs:
     runs-on: "ubuntu-22.04"
 
     steps:
-      - uses: "actions/checkout@v2"
+      - uses: "actions/checkout@v4"
 
       - uses: "shivammathur/setup-php@v2"
         with:
           coverage: none
-          php-version: 8.1
+          php-version: 8.3
           extensions: curl
           tools: composer:v2, cs2pr
 
@@ -34,11 +34,11 @@ jobs:
     runs-on: "ubuntu-22.04"
 
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
 
       - uses: shivammathur/setup-php@v2
         with:
-          php-version: 8.1
+          php-version: 8.3
           extensions: curl
           coverage: none
           tools: composer:v2
@@ -59,11 +59,12 @@ jobs:
     strategy:
       matrix:
         php-version:
-          - "8.0"
           - "8.1"
+          - "8.2"
+          - "8.3"
 
     steps:
-      - uses: "actions/checkout@v2"
+      - uses: "actions/checkout@v4"
 
       - uses: "shivammathur/setup-php@v2"
         with:
@@ -85,11 +86,12 @@ jobs:
     strategy:
       matrix:
         php-version:
-          - "8.0"
           - "8.1"
+          - "8.2"
+          - "8.3"
 
     steps:
-      - uses: "actions/checkout@v2"
+      - uses: "actions/checkout@v4"
 
       - uses: "shivammathur/setup-php@v2"
         with:
@@ -110,11 +112,12 @@ jobs:
     strategy:
       matrix:
         php-version:
-          - "8.0"
           - "8.1"
+          - "8.2"
+          - "8.3"
 
     steps:
-      - uses: "actions/checkout@v2"
+      - uses: "actions/checkout@v4"
 
       - uses: "shivammathur/setup-php@v2"
         with:

diff --git a/composer.json b/composer.json
@@ -31,13 +31,13 @@
         }
     },
     "require": {
-        "php": "^8.0",
-        "dealerdirect/phpcodesniffer-composer-installer": "^0.7.0",
-        "slevomat/coding-standard": "^8.6.4",
+        "php": "^8.1",
+        "dealerdirect/phpcodesniffer-composer-installer": "^0.7 || ^1.0.0",
+        "slevomat/coding-standard": "^8.11",
         "squizlabs/php_codesniffer": "^3.7"
     },
     "require-dev": {
-        "symfony/phpunit-bridge": "^5.1"
+        "symfony/phpunit-bridge": "^6.4"
     },
     "scripts": {
         "test": [

diff --git a/src/Unleashed/Sniffs/Namespaces/FullyQualifiedGlobalFunctionsSniff.php b/src/Unleashed/Sniffs/Namespaces/FullyQualifiedGlobalFunctionsSniff.php
@@ -78,9 +78,13 @@ public function register()
      *
      * @link https://github.com/squizlabs/PHP_CodeSniffer/blob/master/src/Standards/Generic/Sniffs/PHP/ForbiddenFunctionsSniff.php#L118
      *
+     * @param int $stackPtr
+     *
      * {@inheritDoc}
+     *
+     * @phpcsSuppress SlevomatCodingStandard.TypeHints.ParameterTypeHint.MissingNativeTypeHint
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process(File $phpcsFile, $stackPtr): void
     {
         // Abort if we're not in namespaced code
         $firstNamespacePointer = NamespaceHelper::getFirstNamespacePointer($phpcsFile);