Enable CI for PHP 8.2

joomla-framework · Aug 15, 2022 · a3932a4 · a3932a4
1 parent cb87f58
commit a3932a4
Show file tree

Hide file tree

Showing 13 changed files with 69 additions and 25 deletions.
diff --git a/.drone.jsonnet b/.drone.jsonnet
@@ -25,6 +25,7 @@ local composer(phpversion, params) = {
 local phpunit(phpversion) = {
     name: "PHPUnit",
     image: "joomlaprojects/docker-images:php" + phpversion,
+    [if phpversion == "8.2" then "failure"]: "ignore",
     commands: ["vendor/bin/phpunit"]
 };
 
@@ -59,7 +60,7 @@ local pipeline(name, phpversion, params) = {
                 image: "joomlaprojects/docker-images:php7.4",
                 depends: [ "composer" ],
                 commands: [
-                    "vendor/bin/phpcs -p --report=full --extensions=php --standard=PSR12 src/"
+                    "vendor/bin/phpcs --standard=ruleset.xml"
                 ]
             },
             {
@@ -109,5 +110,6 @@ local pipeline(name, phpversion, params) = {
     pipeline("7.3", "7.3", "--prefer-stable"),
     pipeline("7.4", "7.4", "--prefer-stable"),
     pipeline("8.0", "8.0", "--prefer-stable"),
-    pipeline("8.1", "8.1", "--prefer-stable")
+    pipeline("8.1", "8.1", "--prefer-stable"),
+    pipeline("8.2", "8.2", "--prefer-stable --ignore-platform-reqs"),
 ]
diff --git a/.drone.yml b/.drone.yml
@@ -20,7 +20,7 @@
       },
       {
          "commands": [
-            "vendor/bin/phpcs -p --report=full --extensions=php --standard=PSR12 src/"
+            "vendor/bin/phpcs --standard=ruleset.xml"
          ],
          "depends": [
             "composer"
@@ -303,7 +303,44 @@
    ]
 }
 ---
+{
+   "kind": "pipeline",
+   "name": "PHP 8.2",
+   "steps": [
+      {
+         "commands": [
+            "php -v",
+            "composer update --prefer-stable --ignore-platform-reqs"
+         ],
+         "image": "joomlaprojects/docker-images:php8.2",
+         "name": "composer",
+         "volumes": [
+            {
+               "name": "composer-cache",
+               "path": "/tmp/composer-cache"
+            }
+         ]
+      },
+      {
+         "commands": [
+            "vendor/bin/phpunit"
+         ],
+         "failure": "ignore",
+         "image": "joomlaprojects/docker-images:php8.2",
+         "name": "PHPUnit"
+      }
+   ],
+   "volumes": [
+      {
+         "host": {
+            "path": "/tmp/composer-cache"
+         },
+         "name": "composer-cache"
+      }
+   ]
+}
+---
 kind: signature
-hmac: 8ce225386480f94a8ed0cfbb8261489d0bb53a674f4264f92eadfcea27b40ccc
+hmac: adf77e27d980fe75d12a8a5c07da2e0db889d914d4bbf7444b03f4efdd4c0c5e
 
 ...
diff --git a/composer.json b/composer.json
@@ -6,8 +6,8 @@
     "homepage": "https://github.com/joomla-framework/registry",
     "license": "GPL-2.0-or-later",
     "require": {
-        "php": "^7.2.5|^8.0",
-        "joomla/utilities": "^1.4.1|^2.0"
+        "php": "^7.2.5|~8.0.0|~8.1.0",
+        "joomla/utilities": "^2.0"
     },
     "require-dev": {
         "joomla/coding-standards": "^3.0@dev",

diff --git a/ruleset.xml b/ruleset.xml
@@ -1,25 +1,21 @@
 <?xml version="1.0"?>
-<ruleset name="Joomla">
+<ruleset name="PSR12 with Joomla specific additions">
+    <description>
+        This is a PSR-12 compliant ruleset for Joomla.
+    </description>
 
+    <file>src</file>
+    <file>tests</file>
+
+    <arg name="extensions" value="php"/>
     <arg name="report" value="full"/>
     <arg name="tab-width" value="4"/>
     <arg name="encoding" value="utf-8"/>
-    <arg value="sp"/>
     <arg name="colors" />
+    <arg value="sp"/>
 
-    <!-- Exclude folders not containing production code -->
-    <exclude-pattern>*/.github/*</exclude-pattern>
-
-    <!-- Exclude 3rd party libraries. -->
-    <exclude-pattern>*/vendor/*</exclude-pattern>
-
-    <rule ref="Joomla">
-        <exclude name="Generic.Arrays.DisallowShortArraySyntax"/>
-    </rule>
-
-    <rule ref="Joomla.Classes.InstantiateNewClasses">
-        <properties>
-            <property name="shortArraySyntax" value="true"/>
-        </properties>
+    <rule ref="PSR12">
+        <exclude name="Generic.Files.LineLength.MaxExceeded"/>
+        <exclude name="Generic.Files.LineLength.TooLong"/>
     </rule>
 </ruleset>
diff --git a/tests/FactoryTest.php b/tests/FactoryTest.php
@@ -1,4 +1,5 @@
 <?php
+
 /**
  * @copyright  Copyright (C) 2015 Open Source Matters, Inc.
  * @license    GNU General Public License version 2 or later; see LICENSE

diff --git a/tests/Format/IniTest.php b/tests/Format/IniTest.php
@@ -1,4 +1,5 @@
 <?php
+
 /**
  * @copyright  Copyright (C) 2013 Open Source Matters, Inc.
  * @license    GNU General Public License version 2 or later; see LICENSE

diff --git a/tests/Format/JsonTest.php b/tests/Format/JsonTest.php
@@ -1,4 +1,5 @@
 <?php
+
 /**
  * @copyright  Copyright (C) 2013 Open Source Matters, Inc.
  * @license    GNU General Public License version 2 or later; see LICENSE

diff --git a/tests/Format/PhpTest.php b/tests/Format/PhpTest.php
@@ -1,4 +1,5 @@
 <?php
+
 /**
  * @copyright  Copyright (C) 2013 Open Source Matters, Inc.
  * @license    GNU General Public License version 2 or later; see LICENSE

diff --git a/tests/Format/XmlTest.php b/tests/Format/XmlTest.php
@@ -1,4 +1,5 @@
 <?php
+
 /**
  * @copyright  Copyright (C) 2013 Open Source Matters, Inc.
  * @license    GNU General Public License version 2 or later; see LICENSE

diff --git a/tests/Format/YamlTest.php b/tests/Format/YamlTest.php
@@ -1,4 +1,5 @@
 <?php
+
 /**
  * @copyright  Copyright (C) 2013 Open Source Matters,
  * @license    GNU General Public License version 2 or later; see LICENSE

diff --git a/tests/RegistryTest.php b/tests/RegistryTest.php
@@ -1,4 +1,5 @@
 <?php
+
 /**
  * @copyright  Copyright (C) 2013 Open Source Matters, Inc.
  * @license    GNU General Public License version 2 or later; see LICENSE
@@ -706,8 +707,7 @@ public function testTheRegistryCanBeConvertedToAnArray()
                 'foo2'   => 'testtoarray2',
                 'config' => ['foo3' => 'testtoarray3'],
             ],
-            (new Registry(['foo1' => 'testtoarray1', 'foo2' => 'testtoarray2', 'config' => ['foo3' => 'testtoarray3']]
-            ))->toArray(),
+            (new Registry(['foo1' => 'testtoarray1', 'foo2' => 'testtoarray2', 'config' => ['foo3' => 'testtoarray3']]))->toArray(),
             'The Registry should be converted to an array.'
         );
     }

diff --git a/tests/Stubs/Ini.php b/tests/Stubs/Ini.php
@@ -1,4 +1,5 @@
 <?php
+
 /**
  * @copyright  Copyright (C) 2015 Open Source Matters, Inc.
  * @license    GNU General Public License version 2 or later; see LICENSE

diff --git a/tests/Stubs/jregistry.php b/tests/Stubs/jregistry.php
@@ -1,4 +1,6 @@
-<?php /** @noinspection PhpIllegalPsrClassPathInspection */
+<?php
+
+// phpcs:ignoreFile
 
 class JRegistry
 {